changeset:   2479:b42390e29b47
tag:         tip
user:        Guillermo Alvarez de Eulate <guillermo.alvarez@openbravo.com>
date:        Wed Dec 14 16:59:54 2016 +0100
summary:     Fixed issue 34733: Confirmation popups are hidden before destroy them.

diff -r 06badbbc9fbe -r b42390e29b47 web/org.openbravo.mobile.core/source/component/ob-commonbuttons.js
--- a/web/org.openbravo.mobile.core/source/component/ob-commonbuttons.js	Wed Dec 14 14:10:14 2016 +0100
+++ b/web/org.openbravo.mobile.core/source/component/ob-commonbuttons.js	Wed Dec 14 16:59:54 2016 +0100
@@ -308,8 +308,12 @@
   },
   hide: function (args) {
     var resp = true;
+    var executeOBEvents = true;
+    if (args && args.ignoreOBEvents && args.ignoreOBEvents === true) {
+      executeOBEvents = false;
+    }
     this._addArgsToComponent(args);
-    if (this.executeBeforeHide) {
+    if (this.executeBeforeHide && executeOBEvents) {
       resp = this.executeBeforeHide();
       if (_.isUndefined(resp) || _.isNull(resp)) {
         resp = true;
@@ -329,7 +333,7 @@
       }
       // Restore focus keeper
       OB.MobileApp.view.scanningFocus(this.originalScanMode);
-      if (this.executeOnHide) {
+      if (this.executeOnHide && executeOBEvents) {
         this.executeOnHide();
       }
     }
diff -r 06badbbc9fbe -r b42390e29b47 web/org.openbravo.mobile.core/source/utils/ob-utilitiesui.js
--- a/web/org.openbravo.mobile.core/source/utils/ob-utilitiesui.js	Wed Dec 14 14:10:14 2016 +0100
+++ b/web/org.openbravo.mobile.core/source/utils/ob-utilitiesui.js	Wed Dec 14 16:59:54 2016 +0100
@@ -323,6 +323,7 @@
 
       // remove old confirmation box
       for (i = 0; i < components.length; i++) {
+        components[i].hide({ignoreOBEvents: true});
         components[i].destroy();
       }