0040292: "New Order" button is still clickable even if it's disabled

# HG changeset patch
# User Ranjith S R <ranjith@qualiantech.com>
# Date 1551965859 -19800
#      Thu Mar 07 19:07:39 2019 +0530
# Node ID 4a0a42ab968ec09a0254b4b2b689f95d74a5c120
# Parent  88a5c9b52b17a8178dcb80b29903828dc5f4c048
Related to issue 40292 : Tap Event should be prevented for Toolbar New Button in Pay Open Tickets

diff -r 88a5c9b52b17 -r 4a0a42ab968e web/org.openbravo.retail.posterminal/js/pointofsale/view/toolbar-left.js
--- a/web/org.openbravo.retail.posterminal/js/pointofsale/view/toolbar-left.js	Mon Mar 04 18:42:19 2019 -0500
+++ b/web/org.openbravo.retail.posterminal/js/pointofsale/view/toolbar-left.js	Thu Mar 07 19:07:39 2019 +0530
@@ -57,33 +57,38 @@
   },
   processesToListen: ['calculateReceipt', 'addProduct'],
   disableButton: function () {
-    if (!this.model.get('leftColumnViewManager').isMultiOrder()) {
-      this.isEnabled = false;
-      this.setDisabled(true);
-    }
+    this.updateDisabled(true);
   },
   enableButton: function () {
-    if (!this.model.get('leftColumnViewManager').isMultiOrder()) {
-      if (OB.UTIL.isNullOrUndefined(OB.MobileApp.model.get('serviceSearchMode')) || !OB.MobileApp.model.get('serviceSearchMode')) {
-        this.isEnabled = true;
-        this.setDisabled(false);
-      }
+    if (OB.UTIL.isNullOrUndefined(OB.MobileApp.model.get('serviceSearchMode')) || !OB.MobileApp.model.get('serviceSearchMode')) {
+      this.updateDisabled(false);
+    } else {
+      this.updateDisabled(true);
     }
   },
   disabledButton: function (inSender, inEvent) {
-    this.isEnabled = inEvent.disableButtonNew || !inEvent.status;
-    this.setDisabled(inEvent.disableButtonNew || inEvent.status);
+    this.updateDisabled(inEvent.disableButtonNew || inEvent.status);
     if (!this.isEnabled) {
       this.removeClass('btn-icon-new');
     } else {
       this.addClass('btn-icon-new');
     }
   },
+  updateDisabled: function (isDisabled) {
+    if (this.model.get('leftColumnViewManager').isMultiOrder()) {
+      isDisabled = true;
+    }
+    this.isEnabled = !isDisabled;
+    this.setDisabled(isDisabled);
+  },
   init: function (model) {
     this.model = model;
   },
   tap: function () {
     var me = this;
+    if (this.disabled) {
+      return true;
+    }
     OB.UTIL.HookManager.executeHooks('OBPOS_PreCreateNewReceipt', {
       model: this.model,
       context: this