Openbravo Issue Tracking System - Retail Modules
View Issue Details
0036122Retail ModulesWeb POSpublic2017-05-29 11:152017-06-20 09:48
jonibc 
jorge-garcia 
highmajoralways
scheduledopen 
5
 
 
No
0036122: [SERQA 2908]If a partial payment is added, buttons and external input are not blocked.
If a partial payment is added, buttons (UI) and external input are not blocked.

This has special relevance in BUT implementation, since reservations for product are created in the moment that the first payment is done.

The second payment added for an order can not execute the hook until the reservation has been created. It is necessary to take into account that also it is possible to add payments using external input.
Login to WebPOS:

- Open chrome developer tools, put a breakpoint in prePayment hook. (Also it is possible to implement the hook and add a sleep instruction)
- Add a product to the receipt.
- Click on total amount.
- Select cash payment method. Click several times in 10€ button. Check that the hook is raised several times and UI is not blocked.
Block UI and external input while adding payments until each payment has been processed.
No tags attached.
blocks defect 0036010 closed asiermartirena The order and payment is not set in some gift card transactions 
Issue History
2017-05-29 11:15jonibcNew Issue
2017-05-29 11:15jonibcAssigned To => Retail
2017-05-29 11:15jonibcResolution time => 1497218400
2017-05-29 11:15jonibcTriggers an Emergency Pack => No
2017-06-15 12:34jorge-garciaAssigned ToRetail => jorge-garcia
2017-06-15 12:34jorge-garciaStatusnew => acknowledged
2017-06-15 16:48jorge-garciaStatusacknowledged => scheduled
2017-06-16 08:46hgbotCheckin
2017-06-16 08:46hgbotNote Added: 0097426
2017-06-16 08:46hgbotStatusscheduled => resolved
2017-06-16 08:46hgbotResolutionopen => fixed
2017-06-16 08:46hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c92cd921eee5f24927fa74857dc217f2d9cb2edc [^]
2017-06-19 17:09jorge-garciaNote Added: 0097486
2017-06-19 17:09jorge-garciaStatusresolved => new
2017-06-19 17:09jorge-garciaResolutionfixed => open
2017-06-19 17:13jorge-garciaStatusnew => scheduled
2017-06-19 17:40SandrahuguetRelationship addedblocks 0036010
2017-06-20 09:44marvintmResolution time1497218400 =>
2017-06-20 09:44marvintmTypedefect => design defect
2017-06-20 09:46marvintmNote Added: 0097502
2017-06-20 09:48hgbotCheckin
2017-06-20 09:48hgbotNote Added: 0097503

Notes
(0097426)
hgbot   
2017-06-16 08:46   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: c92cd921eee5f24927fa74857dc217f2d9cb2edc
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Jun 15 16:50:40 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c92cd921eee5f24927fa74857dc217f2d9cb2edc [^]

Fixed issue 36122: [SERQA 2908]If a partial payment is added, buttons and
external input are not blocked.

The solution is to show the loading screen when the hook is going to be
executed and hide the screen in the last part of the hook execution.

---
M web/org.openbravo.retail.posterminal/js/model/order.js
---
(0097486)
jorge-garcia   
2017-06-19 17:09   
If a popup is registered inside of preAddPayment hook, the webpos gets stuck in showLoading window.
(0097502)
marvintm   
2017-06-20 09:46   
After analyzing the problem further, we have found that currently there is no reliable way to enforce this which at the same time doesn't impact already existing payment gateway implementations which don't necessarily call the callbackExecutor in every flow.

Therefore, as of now we consider it a design defect. In short we will discuss with all parties to reach a common solution which doesn't negatively impact existing payment providers.
(0097503)
hgbot   
2017-06-20 09:48   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 6f4ce0bdcd19309a7ce62b71593625bdcd105b68
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Jun 20 09:48:11 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6f4ce0bdcd19309a7ce62b71593625bdcd105b68 [^]

Related to issue 36122: [SERQA 2908]If a partial payment is added, buttons and
external input are not blocked.

Backed out changeset c92cd921eee5.

---
M web/org.openbravo.retail.posterminal/js/model/order.js
---