Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0033887Openbravo ERPZ. Otherspublic2016-09-02 12:062016-09-20 02:11
maite 
markmm82 
urgentmajoralways
closedfixed 
5
 
3.0PR16Q4 
aferraz
Advanced Payables and Receivables Mngmt
No
0033887: GL Item accounting dimensions information should be maintained when posting Payment documents
In case Payment document has business partner set in payment's header and GL Item line is included setting different business partner, then accounting entry is generated for header's business partner instead of line's business partner
1. Access Sales Invoice window and select any invoice set as Paid=NO
2. Run "Add payment" process, add glitem line setting different business partner that invoice's partner
3. Access created payment in and run "Post" process. Review Accounting tab and realize that gl item amount is referencing invoice's header business partner instead of business partner set when adding gl item line
accounting dimensions information of gl item lines should be maintained when posting the payment
No tags attached.
Issue History
2016-09-02 12:06maiteNew Issue
2016-09-02 12:06maiteAssigned To => Triage Finance
2016-09-02 12:06maiteModules => Advanced Payables and Receivables Mngmt
2016-09-02 12:06maiteResolution time => 1474063200
2016-09-02 12:06maiteTriggers an Emergency Pack => No
2016-09-02 12:07maiteIssue Monitored: networkb
2016-09-08 10:55aferrazAssigned ToTriage Finance => markmm82
2016-09-08 16:34markmm82Statusnew => scheduled
2016-09-15 16:39markmm82Note Added: 0089991
2016-09-19 12:54hgbotCheckin
2016-09-19 12:54hgbotNote Added: 0090060
2016-09-19 12:54hgbotStatusscheduled => resolved
2016-09-19 12:54hgbotResolutionopen => fixed
2016-09-19 12:54hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/fa17d173e3ce254713c882210c5b6593c399385c [^]
2016-09-19 12:54hgbotCheckin
2016-09-19 12:54hgbotNote Added: 0090061
2016-09-19 12:57aferrazReview Assigned To => aferraz
2016-09-19 12:57aferrazNote Added: 0090062
2016-09-19 12:57aferrazStatusresolved => closed
2016-09-19 12:57aferrazFixed in Version => 3.0PR16Q4
2016-09-20 02:11hudsonbotCheckin
2016-09-20 02:11hudsonbotNote Added: 0090094
2016-09-20 02:11hudsonbotCheckin
2016-09-20 02:11hudsonbotNote Added: 0090095

Notes
(0089991)
markmm82   
2016-09-15 16:39   
1. Access Sales Invoice window and select any invoice set as Paid=NO
2. Run "Add payment" process, add glitem line setting different business partner that invoice's partner
3. Access created payment in and run "Post" process. Review Accounting tab and realize that gl item amount is referencing business partner set when adding gl item line instead of invoice's header business partner
(0090060)
hgbot   
2016-09-19 12:54   
Repository: erp/devel/pi
Changeset: fa17d173e3ce254713c882210c5b6593c399385c
Author: Mark <markmm82 <at> gmail.com>
Date: Wed Sep 14 19:39:08 2016 -0400
URL: http://code.openbravo.com/erp/devel/pi/rev/fa17d173e3ce254713c882210c5b6593c399385c [^]

Fixes issue 33887: GLItem accounting dimensions maintained when posting Payment

In case Payment document has business partner set in payment's header and GL Item line is included setting different business partner, then accounting entry is generated for line's business partner instead of header's business partner.

---
M src/org/openbravo/erpCommon/ad_forms/DocFINPayment.java
---
(0090061)
hgbot   
2016-09-19 12:54   
Repository: erp/devel/pi
Changeset: 7237a9b65b09c6e8faa0939273c998c576e84f03
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Mon Sep 19 12:53:30 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/7237a9b65b09c6e8faa0939273c998c576e84f03 [^]

Related to issue 33887: Code review improvements

- Remove unnecessary parenthesis.
- Add paymentDetail.getFINPaymentScheduleDetailList().get(0).getOrderPaymentSchedule() != null check to avoid NPE.

---
M src/org/openbravo/erpCommon/ad_forms/DocFINPayment.java
---
(0090062)
aferraz   
2016-09-19 12:57   
Code review + Testing OK
(0090094)
hudsonbot   
2016-09-20 02:11   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/3f07553793a6 [^]
Maturity status: Test
(0090095)
hudsonbot   
2016-09-20 02:11   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/3f07553793a6 [^]
Maturity status: Test