Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035320Retail ModulesWeb POSpublic2017-02-21 09:482017-03-09 17:37
Reporteraaroncalero 
Assigned Toranjith_qualiantech_com 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q2 
Merge Request Status
Review Assigned Toguilleaer
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035320: Impossible to create several layaways in a terminal without cash payment methods
DescriptionIn a terminal which does not have any payment method of type cash configured, it is not possible to create two layaways without refreshing the screen.
After creating the first layaway, the "Layaway" button remains disabled.
Steps To ReproduceConfiguration (with White Valley sampledata):
Login in backend, go to the POS Terminal window, find the VBS-1 record and on the Payment Type tab disable the Cash and USA Cash records.
Web pos:
Login in web pos
Create a new layaway (Menu > Layaway this receipt)
Add a product and click on the Total amount button.
Click on Layaway to synchronize the receipt.
Create a second layaway
Add another product and click on the Total amount button.
Now the Layaway button is disabled and a refresh is needed to create a new layaway.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch issue 35320 source posterminal 7868.patch (2,151) 2017-02-27 08:05
https://issues.openbravo.com/file_download.php?file_id=10489&type=bug
patch issue 35320 test.patch (3,391) 2017-02-27 08:05
https://issues.openbravo.com/file_download.php?file_id=10490&type=bug
patch issue35320_2.patch (1,155) 2017-03-07 18:23
https://issues.openbravo.com/file_download.php?file_id=10540&type=bug
Issue History
Date ModifiedUsernameFieldChange
2017-02-21 09:48aaroncaleroNew Issue
2017-02-21 09:48aaroncaleroAssigned To => Retail
2017-02-21 09:48aaroncaleroResolution time => 1488841200
2017-02-21 09:48aaroncaleroTriggers an Emergency Pack => No
2017-02-21 11:29ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2017-02-21 11:29ranjith_qualiantech_comStatusnew => scheduled
2017-02-23 11:35hgbotCheckin
2017-02-23 11:35hgbotNote Added: 0094571
2017-02-23 11:35hgbotCheckin
2017-02-23 11:35hgbotNote Added: 0094572
2017-02-23 11:35hgbotStatusscheduled => resolved
2017-02-23 11:35hgbotResolutionopen => fixed
2017-02-23 11:35hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8ab663ea154199fcd8a36250fd6dd2e87d956152 [^]
2017-02-23 14:17ranjith_qualiantech_comNote Added: 0094577
2017-02-23 14:17ranjith_qualiantech_comStatusresolved => new
2017-02-23 14:17ranjith_qualiantech_comResolutionfixed => open
2017-02-23 14:17ranjith_qualiantech_comStatusnew => scheduled
2017-02-27 08:05ranjith_qualiantech_comFile Added: issue 35320 source posterminal 7868.patch
2017-02-27 08:05ranjith_qualiantech_comFile Added: issue 35320 test.patch
2017-03-01 11:36ranjith_qualiantech_comNote Edited: 0094577bug_revision_view_page.php?bugnote_id=0094577#r14709
2017-03-03 07:14hgbotCheckin
2017-03-03 07:14hgbotNote Added: 0094710
2017-03-03 07:14hgbotStatusscheduled => resolved
2017-03-03 07:14hgbotResolutionopen => fixed
2017-03-03 07:14hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8ab663ea154199fcd8a36250fd6dd2e87d956152 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/65292d7c5078920e25be059e664b7536c1b8fd81 [^]
2017-03-03 07:15hgbotCheckin
2017-03-03 07:15hgbotNote Added: 0094711
2017-03-03 13:33marvintmReview Assigned To => marvintm
2017-03-03 13:33marvintmStatusresolved => closed
2017-03-03 13:33marvintmFixed in Version => RR17Q2
2017-03-07 17:07marvintmNote Added: 0094802
2017-03-07 17:07marvintmStatusclosed => new
2017-03-07 17:07marvintmResolutionfixed => open
2017-03-07 17:07marvintmFixed in VersionRR17Q2 =>
2017-03-07 18:23marvintmFile Added: issue35320_2.patch
2017-03-09 10:08hgbotCheckin
2017-03-09 10:08hgbotNote Added: 0094855
2017-03-09 10:08hgbotStatusnew => resolved
2017-03-09 10:08hgbotResolutionopen => fixed
2017-03-09 10:08hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/65292d7c5078920e25be059e664b7536c1b8fd81 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/78837f4ae39bd44cab57072b31706343771f5fc7 [^]
2017-03-09 17:37guilleaerReview Assigned Tomarvintm => guilleaer
2017-03-09 17:37guilleaerStatusresolved => closed
2017-03-09 17:37guilleaerFixed in Version => RR17Q2

Notes
(0094571)
hgbot   
2017-02-23 11:35   
Repository: tools/automation/pi-mobile
Changeset: da141d714d24699d616d48dced595542e398daae
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Feb 23 16:05:07 2017 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/da141d714d24699d616d48dced595542e398daae [^]

Verifies issue 35320 : Added automated test 'I35320_VerifyLayawayButtonEnabled'

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I35320_VerifyLayawayButtonEnabled.java
---
(0094572)
hgbot   
2017-02-23 11:35   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 8ab663ea154199fcd8a36250fd6dd2e87d956152
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Feb 23 16:05:02 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8ab663ea154199fcd8a36250fd6dd2e87d956152 [^]

Fixes issue 35320 : Layway button should be enabled where receipt is set as layaway

* Code Refactored, Since multiple listener('paymentCancel') created for receipt object in layaway button tap

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/payment.js
---
(0094577)
ranjith_qualiantech_com   
2017-02-23 14:17   
(edited on: 2017-03-01 11:36)
Made it Resolved by mistake, backout the changset due to the repo freeze
(0094710)
hgbot   
2017-03-03 07:14   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 65292d7c5078920e25be059e664b7536c1b8fd81
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Fri Mar 03 11:43:29 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/65292d7c5078920e25be059e664b7536c1b8fd81 [^]

Fixes issue 35320 : Layway button should be enabled where receipt is set as layaway

* Code Refactored, Since multiple listener('paymentCancel') created for receipt object in layaway button tap

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/payment.js
---
(0094711)
hgbot   
2017-03-03 07:15   
Repository: tools/automation/pi-mobile
Changeset: df13c2a543e5ed2ac909affa242dab87620d771a
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Fri Mar 03 11:44:12 2017 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/df13c2a543e5ed2ac909affa242dab87620d771a [^]

Verifies issue 35320 : Added automated test 'I35320_VerifyLayawayButtonEnabled'

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I35320_VerifyLayawayButtonEnabled.java
---
(0094802)
marvintm   
2017-03-07 17:07   
There is one case in which the fix is not enough. If you define the POS Terminal so that it automatically creates layaways, then in the first layaway, the button is enabled, but the next layaway which is created automatically after the first one is completed will have the button disabled.
(0094855)
hgbot   
2017-03-09 10:08   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 78837f4ae39bd44cab57072b31706343771f5fc7
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Tue Mar 07 18:09:04 2017 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/78837f4ae39bd44cab57072b31706343771f5fc7 [^]

Fixed issue 35320. Force update of the layawayAction button in case the ticket itself changed, by checking the docNo

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/payment.js
---