[jira] Commented: (OFBIZ-2720) Add statusId field to the FinAccountTrans entity and verify that existing functionality is not broken.

["David E. Jones (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/OFBIZ-2720?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12729990#action_12729990 ] 

David E. Jones commented on OFBIZ-2720:
---------------------------------------

Could you please comment on the reason for this change, or in other words the business process or use case that lead to it?

The main reason I ask is that this change as described would impact existing installations, which is something we should always avoid if we can (and there are often ways to avoid it). 

If it is the only solution as described here, we need to make sure to update the data migration page:

http://docs.ofbiz.org/display/OFBTECH/Revisions+Requiring+Data+Migration


> Add statusId field to the FinAccountTrans entity and verify that existing functionality is not broken.
> ------------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-2720
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2720
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: accounting
>    Affects Versions: SVN trunk
>            Reporter: Sumit Pandit
>             Fix For: SVN trunk
>
>         Attachments: AddedStatusIdToFinAccountTrans.patch
>
>
> A) Add statusId field to the FinAccountTrans entity.
> B) Seed data: "Created", "Approved".
> C) Create a database upgrade script that sets the statusId to "Approved" to all the records with statusId set to null, no ECA should be triggered when this update is done.
> D) Verify that all the existing code that is using the FinAccountTrans entity works fine; the idea is that if there is code that:
> 1) creates new FinAccountTrans records, then it should create them in the "Approved" status
> 2) if there are accounting transactions posted when FinAccountTrans records are created/updated, this should now happen only when the status is "Approved"
> Thanks Jacopo for the Guidance.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.