ECA and processOrderPayments

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

ECA and processOrderPayments

Jeremy Wick
Hi,

Can someone tell me why the processOrderPayments is not triggered by
cancelOrderItem ? it is by updateOrderItems but cancelOrderItem is
also changing the order right ?

I checked the SVN log and it seems that the following related changes were made:

In Ofbiz 540479 by jacopoc:
The eca was changed so that processOrderPayments is triggered by
resetGrandTotal (instead of updateOrderItems, appendOrderItem,
cancelOrderItem)

Later 546748 the change was partly reversed :
processOrderPayments is no more triggered by resetGrandTotal
(commented, still in trunk like this) but only updateOrderItems and
appendOrderItem were readded.


I am not sure what that means, or if it would cause an issue or not.
But while working on Opentaps which currently has the situation prior
to those two changes i had to patch the processOrderPayments
specifically in regard to cancelOrderItem because the code would try
to approve or complete the order which did not work in case the whole
order was cancelled.

Regards,

--
WICKERSHEIMER
Jérémy