Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031679Retail ModulesWeb POSpublic2015-12-09 18:292016-09-19 19:42
Reporterguilleaer 
Assigned Toshivum039 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR16Q4 
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
Summary0031679: Web POS Crashes when the default payment type is hidden by More button
DescriptionIf a terminal has defined many payment types and default payment type is not shown in the main window (it is shown through more button"), Web POS crashes if other payment type is used and then other payment is added.
Steps To ReproduceERP
Navigate to the terminal and then to the associated terminal type
ensure that field "payment Method" is filled with cash. It means that cash payment type will be used as a default payment type
In terminal type window add a new payment method for credit.
In Terminal window add 3 new payment types for credit payment method. In my example (GAL0, GAL1, GAL2)
Change the lineNo of cash payment type from 10 to 80

Web POS
Login into web POS
Add a product to the ticket
Press pay button
Here you will see that more button is selected by default (green) because default payment method is "inside" more.
Using screen keyboard Press 10€ note and press "enter"
10 will be added for cash
Press card payment type and using screen keyboard Insert an amount (10) and then press enter or press again card button
10 will be added to card
Error 1: press a number and then enter (using screen keyboard) -> An error is shown
Error 2: More button is not green. Default payment type should be selected again
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0031678 closed ranjith_qualiantech_com After pay using a certain payment type, default command changes to scan but default payment type should be selected 
blocks defect 0031675 closed ranjith_qualiantech_com Default Payment method is not correctly selected under certain configuration 
blocks defect 0031677 closed ranjith_qualiantech_com After pay using a certain payment type, default payment type is not re-selected in the UI 
Attached Files? payment_method_issue_4 (4,572,952) 2015-12-09 18:29
https://issues.openbravo.com/file_download.php?file_id=8796&type=bug
? issue31679 (1,075,052) 2016-09-01 22:29
https://issues.openbravo.com/file_download.php?file_id=9772&type=bug
jpg GAL_screenshoot_Selection_448.jpg (502,342) 2016-09-01 22:31
https://issues.openbravo.com/file_download.php?file_id=9773&type=bug
patch 31679 source posterminal.patch (1,312) 2016-09-06 12:54
https://issues.openbravo.com/file_download.php?file_id=9787&type=bug
patch 31679 test modified.patch (1,096) 2016-09-06 12:54
https://issues.openbravo.com/file_download.php?file_id=9788&type=bug
Issue History
Date ModifiedUsernameFieldChange
2015-12-09 18:29guilleaerNew Issue
2015-12-09 18:29guilleaerAssigned To => Retail
2015-12-09 18:29guilleaerFile Added: payment_method_issue_4
2015-12-09 18:29guilleaerTriggers an Emergency Pack => No
2015-12-09 18:36guilleaerRelationship addedblocks 0031675
2015-12-09 18:37guilleaerRelationship addedblocks 0031677
2015-12-09 18:37guilleaerRelationship addedrelated to 0031678
2016-08-08 15:01ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-08-08 15:01ranjith_qualiantech_comStatusnew => scheduled
2016-08-16 14:04hgbotCheckin
2016-08-16 14:04hgbotNote Added: 0089142
2016-08-16 14:04hgbotCheckin
2016-08-16 14:04hgbotNote Added: 0089143
2016-08-16 14:04hgbotStatusscheduled => resolved
2016-08-16 14:04hgbotResolutionopen => fixed
2016-08-16 14:04hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/1cc76413f5ff8f1d8a7939586764b8023cea9c6a [^]
2016-08-17 07:30hgbotCheckin
2016-08-17 07:30hgbotNote Added: 0089163
2016-09-01 22:29guilleaerNote Added: 0089686
2016-09-01 22:29guilleaerStatusresolved => new
2016-09-01 22:29guilleaerResolutionfixed => open
2016-09-01 22:29guilleaerStatusnew => acknowledged
2016-09-01 22:29guilleaerFile Added: issue31679
2016-09-01 22:30guilleaerNote Edited: 0089686bug_revision_view_page.php?bugnote_id=0089686#r13028
2016-09-01 22:31guilleaerFile Added: GAL_screenshoot_Selection_448.jpg
2016-09-02 09:34ranjith_qualiantech_comStatusacknowledged => scheduled
2016-09-06 12:54jorge-garciaFile Added: 31679 source posterminal.patch
2016-09-06 12:54jorge-garciaFile Added: 31679 test modified.patch
2016-09-15 11:18shivum039Assigned Toranjith_qualiantech_com => shivum039
2016-09-16 12:43hgbotCheckin
2016-09-16 12:43hgbotNote Added: 0090037
2016-09-16 12:43hgbotCheckin
2016-09-16 12:43hgbotNote Added: 0090038
2016-09-16 12:43hgbotStatusscheduled => resolved
2016-09-16 12:43hgbotResolutionopen => fixed
2016-09-16 12:43hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/1cc76413f5ff8f1d8a7939586764b8023cea9c6a [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/07ec2d1ad0fcf9b635568320d616b0ee7dc907f1 [^]
2016-09-19 19:42guilleaerReview Assigned To => guilleaer
2016-09-19 19:42guilleaerStatusresolved => closed
2016-09-19 19:42guilleaerFixed in Version => RR16Q4

Notes
(0089142)
hgbot   
2016-08-16 14:04   
Repository: tools/automation/pi-mobile
Changeset: 0ae060691cf3598d025ba94541d3f9c72805c67f
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Aug 16 17:33:49 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/0ae060691cf3598d025ba94541d3f9c72805c67f [^]

Verifies issue 31679 : Added automated test 'I31679_VerifyMultiPaymentMethod'

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I31679_VerifyMultiPaymentMethod.java
---
(0089143)
hgbot   
2016-08-16 14:04   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 1cc76413f5ff8f1d8a7939586764b8023cea9c6a
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Aug 16 17:34:13 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/1cc76413f5ff8f1d8a7939586764b8023cea9c6a [^]

Fixes issue 31679 : Add Popup Payment method commands to keyboard command list

- Keyboard status should not be reset on clicking More button until payment method is selected from popup
- Payment method will be selected in the popup eventhough if it is present in popup

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/keyboard-toolbars.js
---
(0089163)
hgbot   
2016-08-17 07:30   
Repository: tools/automation/pi-mobile
Changeset: 036657939253789f8b0a4a24165f35ea3be9abf2
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Aug 17 11:00:37 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/036657939253789f8b0a4a24165f35ea3be9abf2 [^]

Verifies issue 31679 : Modified test, Removed unused function (Local storage clear)

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I31675_VerifyDefaultCashPaymentMethod.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I31679_VerifyMultiPaymentMethod.java
---
(0089686)
guilleaer   
2016-09-01 22:29   
(edited on: 2016-09-01 22:30)
- A different payment method appears selected (more button is not green) when card is unselected.

See attached video
See attached image with configuration to reproduce the issue
(0090037)
hgbot   
2016-09-16 12:43   
Repository: tools/automation/pi-mobile
Changeset: d3f3f333642cb59c8e28e5e5ae05feb1f00e7aec
Author: Shivanand Madiwalar <shivanand.madiwalar <at> promantia.com>
Date: Fri Sep 16 16:06:23 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/d3f3f333642cb59c8e28e5e5ae05feb1f00e7aec [^]

Verifies Issue 31679 : Web POS Crashes when the default payment type is hidden by More button .

The Provided Solution (Patch) Applied And Tested By Running The Test Created For The Same .

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I31679_VerifyMultiPaymentMethod.java
---
(0090038)
hgbot   
2016-09-16 12:43   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 07ec2d1ad0fcf9b635568320d616b0ee7dc907f1
Author: Shivanand Madiwalar <shivanand.madiwalar <at> promantia.com>
Date: Fri Sep 16 16:06:33 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/07ec2d1ad0fcf9b635568320d616b0ee7dc907f1 [^]

Fixes Issue 31679 : Web POS Crashes when the default payment type is hidden by More button .

The Provided Solution (Patch) Applied And Tested By Running The Test Created For The Same .

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