You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Jacques Le Roux <ja...@les7arts.com> on 2015/04/28 09:16:13 UTC

Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Hi Pranay,

That's cool, but is it not worth a Jira for releases logs?

Thanks

Jacques

Le 24/04/2015 15:29, pranayp@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>
>
>
>

Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Posted by Pranay Pandey <pr...@hotwaxsystems.com>.
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.cappellato@hotwaxsystems.com> wrote:

> Done:
>
> https://issues.apache.org/jira/browse/OFBIZ-6369
>
> Jacopo
>
> On Apr 28, 2015, at 9:16 AM, Jacques Le Roux <ja...@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, pranayp@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>
> >>
> >>
> >>
>
>

Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Posted by Pranay Pandey <pr...@hotwaxsystems.com>.
Thanks Jacques, he is doing better now.

As now you know I wasn't ignoring you, definitely it was worth adding a
JIRA ticket, thanks so much for reminder and persistence.

Best regards,

Pranay Pandey
HotWax Systems
http://www.hotwaxsystems.com/
skype: pranay.pandey
cell: +91 982.603.5576

On Fri, May 15, 2015 at 1:02 PM, Jacques Le Roux <
jacques.le.roux@les7arts.com> wrote:

> Thanks Jacopo :)
>
> And all my wishes to Pranay and his son for a prompt recovery
>
> Jacques
>
>
> Le 15/05/2015 07:09, Jacopo Cappellato a écrit :
>
>> Done:
>>
>> https://issues.apache.org/jira/browse/OFBIZ-6369
>>
>> Jacopo
>>
>> On Apr 28, 2015, at 9:16 AM, Jacques Le Roux <
>> jacques.le.roux@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, pranayp@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>
>>>>
>>>>
>>>>
>>>>
>>
>>

Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Posted by Jacques Le Roux <ja...@les7arts.com>.
Thanks Jacopo :)

And all my wishes to Pranay and his son for a prompt recovery

Jacques

Le 15/05/2015 07:09, Jacopo Cappellato a écrit :
> Done:
>
> https://issues.apache.org/jira/browse/OFBIZ-6369
>
> Jacopo
>
> On Apr 28, 2015, at 9:16 AM, Jacques Le Roux <ja...@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, pranayp@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>
>>>
>>>
>>>
>
>

Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Posted by Jacopo Cappellato <ja...@hotwaxsystems.com>.
Done:

https://issues.apache.org/jira/browse/OFBIZ-6369

Jacopo

On Apr 28, 2015, at 9:16 AM, Jacques Le Roux <ja...@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, pranayp@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>
>> 
>> 
>> 


Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Posted by Divesh Dutta <di...@hotwaxsystems.com>.
Hi Jacques,

As mentioned by Ashish in another thread, Pranay is not actively working
from last few days because his son was admitted to hospital. He will get
back to you Jacques once he is back.

Thanks
--
Divesh Dutta.

On Thu, May 14, 2015 at 3:46 PM, Jacques Le Roux <
jacques.le.roux@les7arts.com> wrote:

> Hi Pranay,
>
> I think you are ignoring me, not sure why though :/ Is it intrusive to ask?
>
> Thanks
>
> Jacques
>
>
> Le 06/05/2015 19:53, Jacques Le Roux a écrit :
>
>> 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, pranayp@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>
>>>>
>>>>
>>>>
>>>>
>>>
>>

Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Posted by Ashish Vijaywargiya <as...@hotwaxsystems.com>.
Pranay is not ignoring you. I posted comment on another thread which
includes details about his unavailability.

Kind Regards
Ashish Vijaywargiya
HotWax Systems - est. 1997

On Thu, May 14, 2015 at 3:46 PM, Jacques Le Roux <
jacques.le.roux@les7arts.com> wrote:

> Hi Pranay,
>
> I think you are ignoring me, not sure why though :/ Is it intrusive to ask?
>
> Thanks
>
> Jacques
>
>
> Le 06/05/2015 19:53, Jacques Le Roux a écrit :
>
>> 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, pranayp@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>
>>>>
>>>>
>>>>
>>>>
>>>
>>

Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Posted by Jacques Le Roux <ja...@les7arts.com>.
Hi Pranay,

I think you are ignoring me, not sure why though :/ Is it intrusive to ask?

Thanks

Jacques

Le 06/05/2015 19:53, Jacques Le Roux a écrit :
> 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, pranayp@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>
>>>
>>>
>>>
>>
>

Re: svn commit: r1675852 - in /ofbiz/trunk/applications/accounting/servicedef: secas.xml secas_invoice.xml

Posted by Jacques Le Roux <ja...@les7arts.com>.
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, pranayp@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>
>>
>>
>>
>