Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0024912
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementmajoralways2013-10-08 18:372013-10-16 19:46
ReporterioritzCiaView Statuspublic 
Assigned Todmiguelez 
PriorityimmediateResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisiond6cb6fc0a752
ProjectionnoneETAnoneTarget Version
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_22
OS VersionUbuntu 10.04Database version8.4Ant version1.7.1
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression levelProduction - Confirmed Stable
Regression date2013-04-25
Regression introduced in release
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/2674db04cea807ccca1b64725f007cd0bf75af26 [^]
Triggers an Emergency PackNo
Summary

0024912: When posting a transaction of a payment with more than one line you get a not balanced error.

DescriptionWhen posting a transaction of a payment with more than one line, the getBalanced method of DocFINFinAcctTransaction.java launches a not balanced error.
Steps To Reproduce- Go to General Ledger Configuration window, General Accounts tab and check that Suspense Balancing Use check is not checked.
- Create a Payment with more than one line and process it.
- Add the Payment to a Transaction and try to post it.
- You get a not balanced error.
Proposed SolutionBigDecimal lineBalance = BigDecimal.ZERO;
for (FIN_PaymentDetail pd : payment.getFINPaymentDetailList()) {
   lineBalance = lineBalance.add(payment.isReceipt() ? pd.getAmount().add(
       pd.getWriteoffAmount()) : pd.getAmount().add(pd.getWriteoffAmount()).negate());
}
retValue = retValue.subtract(lineBalance);

code should be replaced with:

BigDecimal lineBalance = payment.isReceipt() ? new BigDecimal(
    ((DocLine_FINFinAccTransaction) p_lines[i]).getAmount()) : new BigDecimal(
    ((DocLine_FINFinAccTransaction) p_lines[i]).getAmount()).negate();
BigDecimal lineWriteoff = payment.isReceipt() ? new BigDecimal(
    ((DocLine_FINFinAccTransaction) p_lines[i]).getWriteOffAmt()) : new BigDecimal(
    ((DocLine_FINFinAccTransaction) p_lines[i]).getWriteOffAmt()).negate();
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0061636)
hgbot (developer)
2013-10-09 17:04

 Repository: erp/devel/pi
Changeset: d6cb6fc0a7521cc0938d6c3753fae495d8732c4d
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Oct 09 17:04:19 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/d6cb6fc0a7521cc0938d6c3753fae495d8732c4d [^]

Fixes Issue 24912. Prevents from not Balanced error.

---
M src/org/openbravo/erpCommon/ad_forms/DocFINFinAccTransaction.java
---
(0061638)
hgbot (developer)
2013-10-09 17:39

 Repository: erp/devel/pi
Changeset: 81de998abbec582d7cb06fab64989ec37b8098b2
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Oct 09 17:39:29 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/81de998abbec582d7cb06fab64989ec37b8098b2 [^]

Related to Issue 24912.

---
M src/org/openbravo/erpCommon/ad_forms/DocFINFinAccTransaction.java
---
(0061640)
ioritzCia (developer)
2013-10-09 18:23

 Verified
(0061739)
hudsonbot (developer)
2013-10-16 19:46

 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/51315988909a [^]

Maturity status: Test
(0061740)
hudsonbot (developer)
2013-10-16 19:46

 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/51315988909a [^]

Maturity status: Test

- Issue History
Date Modified Username Field Change
2013-10-08 18:37 ioritzCia New Issue
2013-10-08 18:37 ioritzCia Assigned To => dmiguelez
2013-10-08 18:37 ioritzCia Modules => Core
2013-10-08 18:37 ioritzCia Triggers an Emergency Pack => No
2013-10-08 18:39 ioritzCia Regression level => Production - Confirmed Stable
2013-10-08 18:39 ioritzCia Regression date => 2013-04-25
2013-10-08 18:39 ioritzCia Regression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/2674db04cea807ccca1b64725f007cd0bf75af26 [^]
2013-10-09 17:04 hgbot Checkin
2013-10-09 17:04 hgbot Note Added: 0061636
2013-10-09 17:04 hgbot Status new => resolved
2013-10-09 17:04 hgbot Resolution open => fixed
2013-10-09 17:04 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d6cb6fc0a7521cc0938d6c3753fae495d8732c4d [^]
2013-10-09 17:39 hgbot Checkin
2013-10-09 17:39 hgbot Note Added: 0061638
2013-10-09 18:23 ioritzCia Note Added: 0061640
2013-10-09 18:23 ioritzCia Status resolved => closed
2013-10-16 19:46 hudsonbot Checkin
2013-10-16 19:46 hudsonbot Note Added: 0061739
2013-10-16 19:46 hudsonbot Checkin
2013-10-16 19:46 hudsonbot Note Added: 0061740

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker