0041337: The price modification popup is showing over the approval popup

# HG changeset patch
# User Ranjith S R <ranjith@qualiantech.com>
# Date 1563174835 -19800
#      Mon Jul 15 12:43:55 2019 +0530
# Node ID 6e85fcfb84a22cb7a0e2b9249d60519059140850
# Parent  f65a896c19746fde972ac3b0832b3a0e69843ac2
Related to issue 41337 : Approval popup should be closed before callback called.

* If another popup is opened from approved popup callback,
  approval popup should be closed first and then new popup should be opened

diff -r f65a896c1974 -r 6e85fcfb84a2 web/org.openbravo.mobile.core/source/component/obpos-approval.js
--- a/web/org.openbravo.mobile.core/source/component/obpos-approval.js	Thu Jul 11 11:18:32 2019 -0400
+++ b/web/org.openbravo.mobile.core/source/component/obpos-approval.js	Mon Jul 15 12:43:55 2019 +0530
@@ -434,7 +434,8 @@
     var u = this.$.bodyContent.$.username.getValue(),
       p = this.$.bodyContent.$.password.getValue(),
       emptyUserPassword = !u || !p,
-      me = this;
+      me = this,
+      finalCallback;
 
     this.$.bodyContent.$.approvalWarning.setContent('');
     if (emptyUserPassword) {
@@ -444,6 +445,15 @@
       return;
     }
 
+    finalCallback = function(approved, supervisor, approvalType) {
+      me.waterfall('onHideThisPopup', {
+        args: {
+          approvalSent: true
+        }
+      });
+      me.callback(approved, supervisor, approvalType);
+    };
+
     OB.UTIL.checkApproval(
       this.approvalType,
       u,
@@ -465,17 +475,12 @@
               approved,
               supervisor,
               approvalType,
-              me.callback
+              finalCallback
             );
           } else {
-            me.callback(approved, supervisor, approvalType);
+            finalCallback(approved, supervisor, approvalType);
           }
         }
-        me.waterfall('onHideThisPopup', {
-          args: {
-            approvalSent: true
-          }
-        });
       },
       this.model,
       this.attrs