Openbravo Issue Tracking System - Retail Modules
View Issue Details
0034734Retail ModulesWeb POSpublic2016-12-14 17:392016-12-22 17:12
mrecasens 
jorge-garcia 
highmajoralways
closedfixed 
5
 
RR17Q1 
marvintm
No
0034734: Scan of non-existing EAN/UPC product causes loss of next scan
Scanning a barcode that does not correspond to any available product results in a pop up error which is acknowledged inadvertently by any subsequent scan.
This results in the error not being shown and a potentially valid scan for a product to be lost.
0. Go to "POS Terminal Type" window and check "Use External Input" for your terminal, e.g. "VBS POS Terminal Type"

1. Log into Web POS
https://livebuilds.openbravo.com/retail_pi_pgsql/web/org.openbravo.retail.posterminal/?terminal=VBS-1#retail.pointofsale [^]
as vallblanca / openbravo

2. Scan a series of barcodes, one of them shall not correspond to any product:
2.1. scan a valid barcode that corresponds to an EAN of an existing product for the terminal
=> the corresponding article is added to the ticket
2.2. scan a valid barcode that DOES NOT correspond to any EAN of an existing product for the terminal
=> NO article is added to the ticket
=> Instead, a pop up error message is displayed
2.3. scan a valid barcode that corresponds to an EAN of an existing product for the terminal
=> NO article is NOT added to the ticket
=> Instead, the previous pop up error message disappears; the carriage return may have been taken for an acceptation of the error message
2.4 You can continue scanning products

The overall result is that the article in step 2.2 is not added, which may be normal, but neither is the article in step 2.3 which is a perfectly valid article.

A proposed solution would be to disable the scanning of products while the pop error message is raised

ANNEX A
A list of possible EAN codes and their barcodes is provided to help with further testing.
2099901001832
2099901001849
2099901001870
2099901002068
2099901003553
Attached a PDF file with the barcodes above
Further scanning of products should be disabled as long as the pop up error message is present.
​SER QA
causes defect 0036221 closed ebecerra In some of the pop ups of WEBPOS for 17Q2 the focus is not on the first input field 
pdf BUT - Autom EAN for OTF barcodes test.pdf (18,789) 2016-12-14 17:39
https://issues.openbravo.com/file_download.php?file_id=10169&type=bug
Issue History
2016-12-14 17:39mrecasensNew Issue
2016-12-14 17:39mrecasensAssigned To => Retail
2016-12-14 17:39mrecasensFile Added: BUT - Autom EAN for OTF barcodes test.pdf
2016-12-14 17:39mrecasensTriggers an Emergency Pack => No
2016-12-14 17:43mrecasensTag Attached: ​SER QA
2016-12-14 17:44mrecasensIssue Monitored: mrecasens
2016-12-19 12:56marvintmResolution time => 1483052400
2016-12-19 13:53jorge-garciaStatusnew => scheduled
2016-12-19 13:53jorge-garciaAssigned ToRetail => jorge-garcia
2016-12-21 08:42hgbotCheckin
2016-12-21 08:42hgbotNote Added: 0092882
2016-12-21 08:42hgbotCheckin
2016-12-21 08:42hgbotNote Added: 0092883
2016-12-21 08:42hgbotCheckin
2016-12-21 08:42hgbotNote Added: 0092884
2016-12-21 08:42hgbotStatusscheduled => resolved
2016-12-21 08:42hgbotResolutionopen => fixed
2016-12-21 08:42hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/da460c83b37c4b23e9858ea2b7bba4fc929f471d [^]
2016-12-21 08:43hgbotCheckin
2016-12-21 08:43hgbotNote Added: 0092885
2016-12-21 18:44marvintmNote Added: 0092918
2016-12-21 18:44marvintmStatusresolved => new
2016-12-21 18:44marvintmResolutionfixed => open
2016-12-22 10:51jorge-garciaStatusnew => scheduled
2016-12-22 15:17hgbotCheckin
2016-12-22 15:17hgbotNote Added: 0092944
2016-12-22 15:18hgbotCheckin
2016-12-22 15:18hgbotNote Added: 0092945
2016-12-22 15:18hgbotCheckin
2016-12-22 15:18hgbotNote Added: 0092946
2016-12-22 15:19jorge-garciaStatusscheduled => resolved
2016-12-22 15:19jorge-garciaResolutionopen => fixed
2016-12-22 17:12marvintmReview Assigned To => marvintm
2016-12-22 17:12marvintmStatusresolved => closed
2016-12-22 17:12marvintmFixed in Version => RR17Q1
2018-09-20 18:44dmitry_mezentsevRelationship addedcauses 0036221

Notes
(0092882)
hgbot   
2016-12-21 08:42   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: dc2f33eb879b745978cffe6681b27aaa8a29423a
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Mon Dec 19 17:27:23 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/dc2f33eb879b745978cffe6681b27aaa8a29423a [^]

Related to issue 34734: Scan of non-existing EAN/UPC product causes loss of
next scan

In the case the preference OBPOS_KbUPCEANCodeNotFound is active, the popup
shown has not as default action the OK button.

---
M web/org.openbravo.retail.posterminal/js/components/subwindow.js
M web/org.openbravo.retail.posterminal/js/pointofsale/view/keyboardorder.js
---
(0092883)
hgbot   
2016-12-21 08:42   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 637c8a3e72f1fa3dfcaae564fcda22262b4dd243
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Dec 20 12:21:37 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/637c8a3e72f1fa3dfcaae564fcda22262b4dd243 [^]

Related to issue 34734: Scan of non-existing EAN/UPC product causes loss of
next scan

The error sound is played when the product doesn’t exists or the scanMode is
disable in that moment.

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/keyboardorder.js
---
(0092884)
hgbot   
2016-12-21 08:42   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: da460c83b37c4b23e9858ea2b7bba4fc929f471d
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Mon Dec 19 17:21:27 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/da460c83b37c4b23e9858ea2b7bba4fc929f471d [^]

Fixed issue 34734: Scan of non-existing EAN/UPC product causes loss of
next scan

Added the possibility of specify if a confirmation popup has a default action.

If the confirmation button is not set as defaultAction, the focus is not set
in that element.

---
M web/org.openbravo.mobile.core/source/component/ob-commonbuttons.js
M web/org.openbravo.mobile.core/source/utils/ob-utilitiesui.js
---
(0092885)
hgbot   
2016-12-21 08:43   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 651d5c0a549bc4c59be8996a55980dc9ecba8110
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Dec 20 12:19:25 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/651d5c0a549bc4c59be8996a55980dc9ecba8110 [^]

Related to issue 34734: Scan of non-existing EAN/UPC product causes loss of
next scan

Added an error sound to indicate the search of a product has failed. This error
sound can be enable/disable with a preference in the backend.

This preference name is ‘Mobile Reproduce Error Sound On Failed Scan’ and by
default is active.

---
M legal/Licensing.txt
M src-db/database/sourcedata/AD_PREFERENCE.xml
M src-db/database/sourcedata/AD_REF_LIST.xml
M web/org.openbravo.mobile.core/source/component/ob-keyboard.js
A web/org.openbravo.mobile.core/sounds/Computer_Error.mp3
---
(0092918)
marvintm   
2016-12-21 18:44   
I get the error sound in a place not related at all with the scanning action:
- Log in the Web POS
- Open the Address selector.
- Type the "Esc" key. Notice that the error sound is played.
(0092944)
hgbot   
2016-12-22 15:17   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 75a9434301845b0a502ed99e7056a41386022ffa
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Dec 22 10:43:45 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/75a9434301845b0a502ed99e7056a41386022ffa [^]

Related to issue 34734: Scan of non-existing EAN/UPC product causes loss of
next scan

Only the error popup that indicates the EAN/UPC fails must reproduce the error
sound.

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/keyboardorder.js
---
(0092945)
hgbot   
2016-12-22 15:18   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 362ae31579674166d62fb08ccd929de105d49a5d
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Dec 22 10:44:52 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/362ae31579674166d62fb08ccd929de105d49a5d [^]

Related to issue 34734: Scan of non-existing EAN/UPC product causes loss of
next scan

New global variable has been created to indicate if the error sound must be
played if the scan of barcode fails.

---
M web/org.openbravo.mobile.core/source/component/ob-keyboard.js
---
(0092946)
hgbot   
2016-12-22 15:18   
Repository: tools/automation/pi-mobile
Changeset: 47a3363dc6e3472446a825a13579036d8e5f710c
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Dec 22 13:02:05 2016 +0100
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/47a3363dc6e3472446a825a13579036d8e5f710c [^]

Related to issue 34734: Scan of non-existing EAN/UPC product causes loss of
next scan

Change two tests related to RFID to tap on the correct button in
confirmation popu.p

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I32868_VerifyDisableLockPreferenceWork.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/rfid/AddMissingRFIDNumber.java
---