Thanks Jacopo. Best regards,
Pranay Pandey HotWax Systems http://www.hotwaxsystems.com/ skype: pranay.pandey cell: +91 982.603.5576 On Fri, May 15, 2015 at 10:39 AM, Jacopo Cappellato < jacopo.cappell...@hotwaxsystems.com> wrote: > Done: > > https://issues.apache.org/jira/browse/OFBIZ-6369 > > Jacopo > > On Apr 28, 2015, at 9:16 AM, Jacques Le Roux <jacques.le.r...@les7arts.com> > wrote: > > > Hi Pranay, > > > > That's cool, but is it not worth a Jira for releases logs? > > > > Thanks > > > > Jacques > > > > Le 24/04/2015 15:29, pran...@apache.org a écrit : > >> Author: pranayp > >> Date: Fri Apr 24 13:29:56 2015 > >> New Revision: 1675852 > >> > >> URL: http://svn.apache.org/r1675852 > >> Log: > >> Fixed the order in which invoice and payment transactions are created. > >> > >> Payment transactions were being created prior to invoice transactions, > It causes confusion for accountants in real world. > >> > >> It was a seca execution order which was causing the issue on > setInvoiceStatus. Moved the trigger on setInvoiceStatus for > checkInvoicePaymentApplications and capturePaymentsByInvoice from secas.xml > to secas_invoice.xml, so that we do invoice transactions prior to payment. > >> > >> Modified: > >> ofbiz/trunk/applications/accounting/servicedef/secas.xml > >> ofbiz/trunk/applications/accounting/servicedef/secas_invoice.xml > >> > >> Modified: ofbiz/trunk/applications/accounting/servicedef/secas.xml > >> URL: > http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/servicedef/secas.xml?rev=1675852&r1=1675851&r2=1675852&view=diff > >> > ============================================================================== > >> --- ofbiz/trunk/applications/accounting/servicedef/secas.xml (original) > >> +++ ofbiz/trunk/applications/accounting/servicedef/secas.xml Fri Apr 24 > 13:29:56 2015 > >> @@ -158,13 +158,4 @@ under the License. > >> <condition field-name="productTypeId" operator="equals" > value="ASSET_USAGE"/> > >> <action service="createFixedAssetAndLinkToProduct" > mode="sync"/> > >> </eca> > >> - > >> - <eca service="setInvoiceStatus" event="commit"> > >> - <condition field-name="invoiceId" operator="is-not-empty"/> > >> - <condition field-name="statusId" operator="equals" > value="INVOICE_READY"/> > >> - <condition field-name="oldStatusId" operator="not-equals" > value="INVOICE_READY"/> > >> - <condition field-name="oldStatusId" operator="not-equals" > value="INVOICE_PAID"/> > >> - <action service="checkInvoicePaymentApplications" mode="sync"/> > >> - <action service="capturePaymentsByInvoice" mode="sync"/> > >> - </eca> > >> </service-eca> > >> > >> Modified: > ofbiz/trunk/applications/accounting/servicedef/secas_invoice.xml > >> URL: > http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/servicedef/secas_invoice.xml?rev=1675852&r1=1675851&r2=1675852&view=diff > >> > ============================================================================== > >> --- ofbiz/trunk/applications/accounting/servicedef/secas_invoice.xml > (original) > >> +++ ofbiz/trunk/applications/accounting/servicedef/secas_invoice.xml > Fri Apr 24 13:29:56 2015 > >> @@ -47,4 +47,12 @@ under the License. > >> <action service="createMatchingPaymentApplication" > mode="sync"/> > >> </eca> > >> + <eca service="setInvoiceStatus" event="commit"> > >> + <condition field-name="invoiceId" operator="is-not-empty"/> > >> + <condition field-name="statusId" operator="equals" > value="INVOICE_READY"/> > >> + <condition field-name="oldStatusId" operator="not-equals" > value="INVOICE_READY"/> > >> + <condition field-name="oldStatusId" operator="not-equals" > value="INVOICE_PAID"/> > >> + <action service="checkInvoicePaymentApplications" mode="sync"/> > >> + <action service="capturePaymentsByInvoice" mode="sync"/> > >> + </eca> > >> </service-eca> > >> > >> > >> > >