What are best practice to calculate total property of an aggregate ? about node-cqrs-domain HOT 16 CLOSED

The important part is to manipulate the aggregate only in the event handling part.
But it's ok to calculate it this way and write the result in the event... shouldn't be a problem.

from node-cqrs-domain.

Yes I'm agree with you it should work but it doesn't work.

In the invoiceLineAdded event I push my line into the array "lines" and I read it in the function updateTotal. But this code is not sync, I pass in updateTotal before invoiceLineAddedEvent. So I've always a wrong value in my total.

from node-cqrs-domain.

Can you make a gist or paste all relevant parts (cmdHandle, evtHandle, etc...)?

from node-cqrs-domain.

You've got it here https://gist.github.com/sbiaudet/253e9f53b3ac76235d35

from node-cqrs-domain.

Why has calculateTotal to be async?

from node-cqrs-domain.

You talk about async.parallel ? this code could be different and better, but it's not the problem here. I'v added some comments with number to explicit the exec order.

from node-cqrs-domain.

are the number the desired exec order? what is the current exec order?

from node-cqrs-domain.

It’s the current order.

from node-cqrs-domain.

and how it's the desired exec order?

from node-cqrs-domain.

I put it the desired order.

Le 27 oct. 2014 à 13:51, Adriano Raiano [email protected] a écrit :

and how it's the desired exec order?

—
Reply to this email directly or view it on GitHub #27 (comment).

from node-cqrs-domain.

ok, the problem is that the apply function in the command part will just define a new event but it will apply all events after the command handle part... https://github.com/adrai/node-cqrs-domain/blob/master/lib/definitions/aggregate.js#L498

from node-cqrs-domain.

Yes I see this code. I've make some test before wrote this post. Have you got an elegant solution ? How do you manage aggregate root in you project ? Is it better to send all children entities in one command (lineChanged for ex) or have one command per action ?
I'd like to have your expertise to have an elegant code .

from node-cqrs-domain.

I have an idea... wait a moment....

from node-cqrs-domain.

Can you try the master?

from node-cqrs-domain.

Yes !!! it's work fine. All unit tests passed.

thank's

from node-cqrs-domain.

ok... will release... in a sec

from node-cqrs-domain.