Openbravo Issue Tracking System - Retail Modules
View Issue Details
0034586Retail ModulesWeb POSpublic2016-11-18 15:052016-12-07 14:54
malsasua 
marvintm 
highmajoralways
closedfixed 
5
 
RR17Q1 
guilleaer
No
0034586: it is posible to create receipts related with processed cashup
there is one way to create receipts from WEBPOS related with processed cashups
in webpos:
. create receipt1 associated to cashup1 (it is synced successfully)
. process the cashup (it is synced, but it is stored in "Data Import Entry" in error status)
-> new cashup (with id cashup2) has been created in terminal but not synced with backoffice
. clean the cache of terminal
the cashup loaded from backend is the cashup live in backoffice (cashup1)
. do a receipt (receipt2 associated with cashup1 is synced and stored in Data Import Entry with status Error, waiting to process the cashup created in step1)


in backoffice:
. fix the error of cashup1 and process it
. process receipt2 -> error: the cashup associated to this receipt has been processed
No tags attached.
causes defect 0035367RR17Q2 closed marvintm Not possible to log in the Web POS if there are resolved errors in the Errors While Importing POS data window 
Issue History
2016-11-18 15:05malsasuaNew Issue
2016-11-18 15:05malsasuaAssigned To => Retail
2016-11-18 15:05malsasuaResolution time => 1481324400
2016-11-18 15:05malsasuaTriggers an Emergency Pack => No
2016-12-06 13:14marvintmStatusnew => scheduled
2016-12-06 13:14marvintmAssigned ToRetail => marvintm
2016-12-06 19:27hgbotCheckin
2016-12-06 19:27hgbotNote Added: 0092145
2016-12-06 19:27hgbotStatusscheduled => resolved
2016-12-06 19:27hgbotResolutionopen => fixed
2016-12-06 19:27hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/02c6c8e2af1710fd1ca9e865b55ed7996fe6151a [^]
2016-12-06 20:35hgbotCheckin
2016-12-06 20:35hgbotNote Added: 0092148
2016-12-06 20:35hgbotCheckin
2016-12-06 20:35hgbotNote Added: 0092149
2016-12-07 14:44hgbotCheckin
2016-12-07 14:44hgbotNote Added: 0092195
2016-12-07 14:54guilleaerReview Assigned To => guilleaer
2016-12-07 14:54guilleaerStatusresolved => closed
2016-12-07 14:54guilleaerFixed in Version => RR17Q1
2017-02-27 12:34marvintmRelationship addedcauses 0035367

Notes
(0092145)
hgbot   
2016-12-06 19:27   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 02c6c8e2af1710fd1ca9e865b55ed7996fe6151a
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Tue Dec 06 15:59:52 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/02c6c8e2af1710fd1ca9e865b55ed7996fe6151a [^]

Fixed issue 34586. Restrict login in case cache is lost, and there is an error in a cashup.
- If a processed cashup fails with error (either as importentry, or later on as a POS error), then its corresponding record in the backend is not marked as complete.
- This is not a problem in the case the Web POS continues to work normally, because a new cashup is created automatically in the client side.
- However, if the cache is then cleared, then currently there is a problem, because the Web POS tries to load the current cashup from the backend, and here the record was not updated yet, because of the error.
- The solution is to detect this case, and then restrict the login in this terminal until the problem is resolved.

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M src/org/openbravo/retail/posterminal/master/Cashup.java
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
M web/org.openbravo.retail.posterminal/js/utils/cashUpReportUtils.js
---
(0092148)
hgbot   
2016-12-06 20:35   
Repository: tools/automation/pi-mobile
Changeset: 2492d5cadf781752fe51afafed4fe94e31583ae5
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Tue Dec 06 16:01:20 2016 +0100
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/2492d5cadf781752fe51afafed4fe94e31583ae5 [^]

Related to issue 34586. Added automated test for this case.

---
M src-test/org/openbravo/test/mobile/core/utils/AllowedErrorsHelper.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I34586_CashupErrorLoginRestricted.java
---
(0092149)
hgbot   
2016-12-06 20:35   
Repository: tools/automation/pi-mobile
Changeset: d4dd0921b0c321e9e5f2fd881d8702137996fc62
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Tue Dec 06 17:44:13 2016 +0100
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/d4dd0921b0c321e9e5f2fd881d8702137996fc62 [^]

Related to issue 34586. Removed useless semicolon

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I34586_CashupErrorLoginRestricted.java
---
(0092195)
hgbot   
2016-12-07 14:44   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 89bc41caf8f86fe884ab397d31b8c3ade23e044c
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Wed Dec 07 14:37:55 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/89bc41caf8f86fe884ab397d31b8c3ade23e044c [^]

Related to issue 34586. Ensure code doesn't fail if data is null

---
M web/org.openbravo.retail.posterminal/js/utils/cashUpReportUtils.js
---