Details
-
Type: Task
-
Status: Open
-
Priority: Trivial
-
Resolution: Unresolved
-
Affects Version/s: 4.7.16
-
Fix Version/s: Unscheduled
-
Component/s: CiviContribute
-
Labels:None
-
Versioning Impact:None (no code merged)
-
Documentation Required?:Developer Doc
-
Funding Source:Contributed Code
-
Verified?:No
Description
Pradeep, you wrote in an issue today that the spec for changes to Payment Instrument and to Financial Account was to reverse out old transaction and put in new. The documentation says that for Financial Account, but not for Payment Instrument (https://wiki.civicrm.org/confluence/pages/viewpage.action?pageId=390955013#CiviAccountsDataFlow-ChangePaymentInstrumentforaPartlyPaidorCompletedcontribution). I recall you saying years ago that you did not implement this as per spec but did reversal transactions instead in this situation. Could you confirm via a test that the system is actually producing the reversal transactions, and if so then update the documentation to what is built and running, which from a bookkeeping perspective is equivalent.