Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034976Retail ModulesWeb POSpublic2016-05-17 18:072017-03-08 19:22
ReporterVictorVillar 
Assigned Toguilleaer 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionRR17Q1Fixed in VersionRR17Q1 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034976: [SER QA 969] Search popups in web pos (Receipts/BPs/Locations) stop working if they are opened and the the pos window is resized
DescriptionWhen a search popup is opened in webpos (e.g. the Receipts popup), then dismissed, and then the browser is resized, the next time the same popup is opened, it won't work.
The html element of the search popup will appear duplicated
Steps To ReproduceLogin in web pos.
Open the Receipts search and verify it works.
Close the popup.
Resize the browser.
Open the Receipts search popup again.
The popup doesn't work anymore.
Verify that there are two html divs representing two different instances of the same popup.

Attached video: https://drive.google.com/a/openbravo.com/file/d/0B3RBrE5TUxwbVjJ5RVlfR3B6VzA/view [^]

The workaround is press F5.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks design defect 0032975 closed guilleaer [SER QA 969] Search popups in web pos (Receipts/BPs/Locations) stop working if they are opened and the the pos window is resized 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-01-17 08:54guilleaerTypedesign defect => backport
2017-01-17 08:54guilleaerTarget Version => RR17Q1
2017-01-18 10:25hgbotCheckin
2017-01-18 10:25hgbotNote Added: 0093679
2017-01-18 10:25hgbotStatusscheduled => resolved
2017-01-18 10:25hgbotResolutionopen => fixed
2017-01-18 10:25hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/f64a3739162d6077d00b3f9ae3fa78aa870650f7 [^] => http://code.openbravo.com/retail/backports/3.0RR17Q1/org.openbravo.mobile.core/rev/bd8528492ee990241e14e1b8a3f36e97189a9636 [^]
2017-01-18 16:58hgbotCheckin
2017-01-18 16:58hgbotNote Added: 0093707
2017-03-08 19:22marvintmReview Assigned To => marvintm
2017-03-08 19:22marvintmStatusresolved => closed
2017-03-08 19:22marvintmFixed in Version => RR17Q1

Notes
(0093679)
hgbot   
2017-01-18 10:25   
Repository: retail/backports/3.0RR17Q1/org.openbravo.mobile.core
Changeset: bd8528492ee990241e14e1b8a3f36e97189a9636
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Wed Jan 18 10:07:14 2017 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR17Q1/org.openbravo.mobile.core/rev/bd8528492ee990241e14e1b8a3f36e97189a9636 [^]

Fixed issue 34976: Popups will continue working even after a resize event

This fix have 2 parts:
1st part: When a popup is redered (included into the DOM) a list of rendered popups is managed. This is done in the render function of OB.UI.Popup component
2nd part: When a resize event happens, we will iterate the list of rendered popups (rendered means that it is part of the DOM, not thet it is being shown). If the popup is being shown we will hide it and we will destroy the DOM element. Destroying it, the render process which is executed after resize will work fine and no duplicates will be created in the DOM

---
M web/org.openbravo.mobile.core/source/component/ob-commonbuttons.js
M web/org.openbravo.mobile.core/source/component/ob-terminal-component.js
---
(0093707)
hgbot   
2017-01-18 16:58   
Repository: retail/backports/3.0RR17Q1/org.openbravo.mobile.core
Changeset: af3cd29833f1e9b58e3b174397988aed6a558544
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Wed Jan 18 16:58:08 2017 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR17Q1/org.openbravo.mobile.core/rev/af3cd29833f1e9b58e3b174397988aed6a558544 [^]

Related to issue 34976: update copyright year

---
M web/org.openbravo.mobile.core/source/component/ob-commonbuttons.js
M web/org.openbravo.mobile.core/source/component/ob-terminal-component.js
---