Hi Pranay,

Any chances, or is it out of subject?

Jacques

Le 28/04/2015 09:16, Jacques Le Roux a écrit :
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>




Reply via email to