Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0032875Retail ModulesCash uppublic2016-05-05 14:242016-07-15 15:05
Reportermalsasua 
Assigned Tojorge-garcia 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0032875: transactions are not reconciled by the cashup
Descriptionin some cases, working offline, some transactions are not reconciled by the cashup.
Steps To Reproducein terminal:
1. do cashup (to clean)
2. do a receipt (P1)
(sync successfull)
-- go offline
3. do a receipt (P2)
4. process cashup
5. do a receipt (P3)
-- go online
press F5 -> sync successfull
6. do a receipt (P4)
7. process cashup

>> the transaction related to P3 has not been included in reconciliation created by cashup in step 7

I have reproduced it in RR16Q1, RR15Q3 and RR15Q2, but not in RR15Q1
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-05-05 14:24malsasuaNew Issue
2016-05-05 14:24malsasuaAssigned To => Retail
2016-05-05 14:24malsasuaResolution time => 1465336800
2016-05-05 14:24malsasuaTriggers an Emergency Pack => No
2016-05-10 10:41OrekariaNote Added: 0086337
2016-05-10 10:41OrekariaStatusnew => feedback
2016-05-10 10:41OrekariaAssigned ToRetail => malsasua
2016-05-10 12:19malsasuaNote Added: 0086348
2016-05-10 12:19malsasuaAssigned Tomalsasua => Orekaria
2016-05-10 12:19malsasuaStatusfeedback => new
2016-05-12 14:41OrekariaStatusnew => acknowledged
2016-05-12 14:42OrekariaAssigned ToOrekaria => Retail
2016-05-24 12:52jorge-garciaStatusacknowledged => scheduled
2016-05-24 12:52jorge-garciaAssigned ToRetail => jorge-garcia
2016-07-15 12:33hgbotCheckin
2016-07-15 12:33hgbotNote Added: 0088521
2016-07-15 12:33hgbotCheckin
2016-07-15 12:33hgbotNote Added: 0088523
2016-07-15 12:34hgbotCheckin
2016-07-15 12:34hgbotNote Added: 0088525
2016-07-15 12:34hgbotStatusscheduled => resolved
2016-07-15 12:34hgbotResolutionopen => fixed
2016-07-15 12:34hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8ee815642094928b0fa2bc661160011f33fae1b5 [^]
2016-07-15 15:01hgbotCheckin
2016-07-15 15:01hgbotNote Added: 0088546
2016-07-15 15:05marvintmReview Assigned To => marvintm
2016-07-15 15:05marvintmStatusresolved => closed

Notes
(0086337)
Orekaria   
2016-05-10 10:41   
What means some cases? 1/1000, 1/10000?

Is there a context in which this happens? Which is the error shown (logclient message)?
(0086348)
malsasua   
2016-05-10 12:19   
"some cases" means that it does not happen always, it happens with the "steps to reproduce" informed. In the reproducibility field, it is Always, so with this steps, you can reproduce it always.

You can check it in livebuilds, it is not necessary any special context
(0088521)
hgbot   
2016-07-15 12:33   
Repository: tools/automation/pi-mobile
Changeset: 0c447c8d6c61a072ffd8a10ffef343f9e9908bd7
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jul 15 11:42:17 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/0c447c8d6c61a072ffd8a10ffef343f9e9908bd7 [^]

Verifies issue 33172, Verifies issue 32875: CashUp changes

Change some tests to adjust to the new functionality.

---
M src-test/org/openbravo/test/mobile/common/selenium/utils/OBOfflineUtils.java
M src-test/org/openbravo/test/mobile/retail/extmodules/selenium/tests/cashup/I32177_VerifyInitCashupAfterCashup_II.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/I32177_VerifyInitCashupAfterCashup.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOfflineCashup.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOfflineCashupWait.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOfflineSimplesale.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOfflineSimplesaleWait.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOnlineCashup.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOnlineCashupWait.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I30962_CheckUpdateModalBeingProcessed.java
---
(0088523)
hgbot   
2016-07-15 12:33   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 13825ee93af9b930df7da27a3f23d41d1a21c3fc
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jul 15 11:44:44 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/13825ee93af9b930df7da27a3f23d41d1a21c3fc [^]

Related to issue 33172, Related to issue 32875: CashUp changes

Change synchronization criterias of the CashUp model.

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-modalmodelstosync.js
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
---
(0088525)
hgbot   
2016-07-15 12:34   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 8ee815642094928b0fa2bc661160011f33fae1b5
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jul 15 12:31:13 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8ee815642094928b0fa2bc661160011f33fae1b5 [^]

Fixed issue 33172, Fixed issue 32875: CashUp changes

The CashUp must be the last model to be synchronized in the backend. Sometimes,
this is not the case, and some synchronization models (such as Order and
CashMgmt) are not related with any cashup.

The solution, is to send the cashUp report within the synchronization models
(Order, CashMgmt, VoidLayaway and the CashUp itself).

---
M src-db/database/model/tables/OBPOS_APP_CASHUP.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src/org/openbravo/retail/posterminal/OrderLoader.java
M src/org/openbravo/retail/posterminal/ProcessCashClose.java
M src/org/openbravo/retail/posterminal/ProcessCashMgmt.java
M src/org/openbravo/retail/posterminal/ProcessVoidLayaway.java
M web/org.openbravo.retail.posterminal/js/cashmgmt/model/cashmgmt-model.js
M web/org.openbravo.retail.posterminal/js/closecash/model/cashup-model.js
M web/org.openbravo.retail.posterminal/js/data/dataordersave.js
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
M web/org.openbravo.retail.posterminal/js/model/cashmanagement.js
M web/org.openbravo.retail.posterminal/js/pointofsale/model/pointofsale-model.js
M web/org.openbravo.retail.posterminal/js/utils/cashManagementUtils.js
M web/org.openbravo.retail.posterminal/js/utils/cashUpReportUtils.js
A src/org/openbravo/retail/posterminal/UpdateCashup.java
---
(0088546)
hgbot   
2016-07-15 15:01   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: b99523e5ae3d7998ba6760979fd3933fe228da79
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jul 15 15:00:55 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/b99523e5ae3d7998ba6760979fd3933fe228da79 [^]

Related to issue 33172, Related to issue 32875: CashUp changes

Remove some code that is commented and set a proper year license.

---
M src/org/openbravo/retail/posterminal/UpdateCashup.java
M web/org.openbravo.retail.posterminal/js/data/dataordersave.js
---