Openbravo Issue Tracking System - Retail Modules
View Issue Details
0033802Retail ModulesWeb POSpublic2016-08-25 12:282016-08-29 10:34
xabiermerino 
mario_castello 
urgentmajoralways
closedfixed 
5
 
 
migueldejuana
No
0033802: [SERQA 1656] Button 'done' does not appear when you void a layaway
When you configure a terminal without payment methods and try to void a layaway the button 'Done' (that finish the process) does not appear.
- Go to backend and go to POS Terminal window
- Select VBS1004 terminal and go to Payment Method tab
- Deactivate all payment methods
- Go to the WebPOS VBS1004
- Create a layaway and confirm it clicking on the button Layaway
- Search the layaway
- Click on menu entry 'void this layaway'
- Realize that the button Done does not appear, the user is blocked
SER-QA
Issue History
2016-08-25 12:28xabiermerinoNew Issue
2016-08-25 12:28xabiermerinoAssigned To => Retail
2016-08-25 12:28xabiermerinoResolution time => 1473285600
2016-08-25 12:28xabiermerinoTriggers an Emergency Pack => No
2016-08-25 12:28xabiermerinoTag Attached: SER-QA
2016-08-25 13:10guilleaerNote Added: 0089392
2016-08-25 18:00mario_castelloAssigned ToRetail => mario_castello
2016-08-25 22:08mario_castelloStatusnew => acknowledged
2016-08-25 22:09mario_castelloStatusacknowledged => scheduled
2016-08-26 16:32hgbotCheckin
2016-08-26 16:32hgbotNote Added: 0089469
2016-08-26 16:32hgbotStatusscheduled => resolved
2016-08-26 16:32hgbotResolutionopen => fixed
2016-08-26 16:32hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/70b271ebc3aad3e3cbdd4036e0c3b6a86bb532e0 [^]
2016-08-27 02:10hgbotCheckin
2016-08-27 02:10hgbotNote Added: 0089474
2016-08-29 10:34migueldejuanaReview Assigned To => migueldejuana
2016-08-29 10:34migueldejuanaNote Added: 0089485
2016-08-29 10:34migueldejuanaStatusresolved => closed

Notes
(0089392)
guilleaer   
2016-08-25 13:10   
This action just should be allowed when the layaway has not payments.

Be careful while doing the fix since this function is used from multiple flows.

Here is the code which should be checked:
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/94dca5a2d2fa/web/org.openbravo.retail.posterminal/js/pointofsale/view/payment.js#l361 [^]
(0089469)
hgbot   
2016-08-26 16:32   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 70b271ebc3aad3e3cbdd4036e0c3b6a86bb532e0
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Thu Aug 25 14:19:12 2016 -0600
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/70b271ebc3aad3e3cbdd4036e0c3b6a86bb532e0 [^]

Fixed issue 33802: Added condition to show done button if the receipt is a void layaway

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/payment.js
---
(0089474)
hgbot   
2016-08-27 02:10   
Repository: tools/automation/pi-mobile
Changeset: ca2a35df60efc3bf93c43801b605b03978dac9aa
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Fri Aug 26 18:09:47 2016 -0600
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/ca2a35df60efc3bf93c43801b605b03978dac9aa [^]

Related to issue 33802: Added automated test

---
A src-test/org/openbravo/test/mobile/quarantine/pack/MCA/layaway/I33802_VoidLayawaywithnotPaymenthMethod.java
---
(0089485)
migueldejuana   
2016-08-29 10:34   
Tested and reviewed