Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0027394Retail ModulesWeb POSpublic2014-08-20 17:312014-09-05 11:46
Reporterjecharri 
Assigned Tomarvintm 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product VersionRR14Q2.1 
Target VersionFixed in VersionRR14Q4 
Merge Request Status
Review Assigned ToOrekaria
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0027394: Cash up process doesn't work correctly in case a reconciliation in draft status exists in the system
DescriptionIf the cash up process finds that there is a reconciliation in draft status, it fails with an error.

The problem is that the way this error is generated is not handled correctly. Instead of logging the cash up as an error, the backend class fails, and the error reaches the client.

It's not clear that this behavior happens in Q3, but it can be reproduced at least in Q2.
Steps To Reproduce- Create a reconciliation in draft status.
- Execute the cash up process.
- Verify that it fails with a weird recursive error in the backend, and that the cash up is not logged as an error in the Errors while importing POS data window.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0027225RR15Q1 closed marvintm ProcessCashClose: Infinite loop/stackoverflow when there are reconcilliations in draft mode 
has duplicate defect 0026942 closed marvintm Cash up process doesn't work correctly in case a reconciliation in draft status exists in the system 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-08-20 17:31jecharriNew Issue
2014-08-20 17:31jecharriAssigned To => marvintm
2014-08-20 17:31jecharriResolution time => 1411509600
2014-08-20 17:31jecharriTriggers an Emergency Pack => No
2014-08-20 17:31jecharriIssue generated from0026942
2014-08-21 01:55eintelauIssue Monitored: eintelau
2014-08-29 15:02mtaalRelationship addedrelated to 0027225
2014-08-29 15:02mtaalNote Added: 0069857
2014-08-29 15:03mtaalRelationship addedhas duplicate 0026942
2014-08-29 15:05hgbotCheckin
2014-08-29 15:05hgbotNote Added: 0069858
2014-08-29 15:05hgbotStatusnew => resolved
2014-08-29 15:05hgbotResolutionopen => fixed
2014-08-29 15:05hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/51e0efd8aea41ff951d43f363504d58ce5e5a7b2 [^]
2014-09-05 11:46OrekariaReview Assigned To => Orekaria
2014-09-05 11:46OrekariaStatusresolved => closed
2014-09-05 11:46OrekariaFixed in Version => RR14Q4

Notes
(0069857)
mtaal   
2014-08-29 15:02   
For q2: the 27225 issue has a patch which will work in the q2 version. It has been applied and tested with Grape.
(0069858)
hgbot   
2014-08-29 15:05   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 51e0efd8aea41ff951d43f363504d58ce5e5a7b2
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Fri Aug 29 15:04:40 2014 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/51e0efd8aea41ff951d43f363504d58ce5e5a7b2 [^]

Fixes issue 27394: Cash up process doesn't work correctly in case a reconciliation in draft status exists in the system
Throw an exception which ends up in the error window

---
M src/org/openbravo/retail/posterminal/ProcessCashClose.java
---