[
https://issues.apache.org/jira/browse/OFBIZ-2720?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12730324#action_12730324
]
Jacopo Cappellato commented on OFBIZ-2720:
------------------------------------------
Hi David,
I agree we should address backward compatibility as much as possible (and step
C was about creating an upgrade script intended to be referenced from the
confluence page you mentioned).
By the way, here are some notes about the use case and design notes: the idea
is to allow the maintenance of data related to the Bank account(s) of the
company. We are thinking to use FinAccount to represent bank accounts. Users
should be able to associate payments (incoming and outgoing) to the bank
account (using FinAccountTrans entries). Payments can be associated to the bank
account in any status (also when they are paid) and initially the
FinAccountTrans entries are created in the "Created" status.
At a later time, when the bank send the official bank statement of the account,
the entries are 'reconciled' and marked as "Approved".
This will also trigger, if setup is done properly, a special GL process to
'move' (i.e. thru the creation of an opposite D/C GL trans entry) the payment
transactions from the GL accounts in which they are to the GL account that
represents the "bank asset account" (FinAccount.postToGlAccount)... but we can
probably discuss this at a later point.
> 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.
