0025147: CashUpReport reports incorrect To keep and To deposit amounts on multicurrency POS terminal

Description

When configuring a POS Terminal with multiple Payment Types such as EUR cash and USD cash the report fails to report the correct amounts 'to keep' and amounts 'to deposit'.
More precisely the amounts for the first currency that was processed are correct, the amounts for subsequent Payment Types are messed up.

Steps To Reproduce

See the description.

Proposed Solution

There is a bug in CashUpReport.java: the BigDecimal 'cashToDeposit' variable is not reset to zero when looping to the next currency.
Consequently it sums up amounts from different currencies starting from the second currency that is dealt with.
Solution: this variable should be defined within the loop and retinitialised each time the loop code is run. The attached patched file corrected the bug for us.

Tags

No tags attached.

Attached Files

CashUpReport.java [^] (28,918 bytes) 2013-11-13 13:14

Relationships [ Relation Graph ] [ Dependency Graph ]

duplicate of

defect

0024911

RMP29

closed

malsasua

Probelms with cash up report in the ERP

Notes
There are no notes attached to this issue.

Issue History
Date Modified	Username	Field	Change
2013-11-13 13:14	mekom	New Issue
2013-11-13 13:14	mekom	Assigned To	=> marvintm
2013-11-13 13:14	mekom	File Added: CashUpReport.java
2013-11-13 13:14	mekom	Triggers an Emergency Pack	=> No
2014-07-17 13:18	adrianromero	Review Assigned To	=> adrianromero
2014-07-17 13:18	adrianromero	Relationship added	duplicate of 0024911
2014-07-17 13:18	adrianromero	Status	new => closed
2014-07-17 13:18	adrianromero	Resolution	open => duplicate
2014-07-17 13:18	adrianromero	Fixed in Version	=> RMP29