0035766: External Keyboard duplicate last character in iOS

# HG changeset patch
# User Antonio Moreno <antonio.moreno@openbravo.com>
# Date 1491993724 -7200
#      Wed Apr 12 12:42:04 2017 +0200
# Node ID b2d48f7ed8100bf3fb5d8f9a91ff7190465fbdb7
# Parent  743d566c0db42fa8378d8d7b63dfee765c62fcce
Fixed issue 35766. Two main changes have been made:
- Now in iOS, the focus keeper value will no longer be reset with the value of the editbox. This is necessary to avoid a problem which might happen, in which the last characters are duplicated.
- Additionally, also only on iOS, a small timeout has been added only after the enter key is pressed. This is necessary to avoid a problem which consisted on the Web POS actually executing the scan action too quickly and missing the last characters in the code.
- Besides, the enterEvent is now called iosKeyEvent, as it's only used in iOS, and we are now only adding the keypress event in iOS and the keyup only in the rest of the platforms. The end result should be the same as before, because previously we were just cancelling the keyup event in iOS, but performance should be a bit better.

diff -r 743d566c0db4 -r b2d48f7ed810 web/org.openbravo.mobile.core/source/component/ob-keyboard.js
--- a/web/org.openbravo.mobile.core/source/component/ob-keyboard.js	Wed Apr 12 11:29:47 2017 +0200
+++ b/web/org.openbravo.mobile.core/source/component/ob-keyboard.js	Wed Apr 12 12:42:04 2017 +0200
@@ -528,7 +528,9 @@
       }
     }
     this.$.editbox.adjustFontSize();
-    document.getElementById('_focusKeeper').value = this.$.editbox.getContent();
+    if (!OB.UTIL.isIOS()) {
+      document.getElementById('_focusKeeper').value = this.$.editbox.getContent();
+    }
   },
 
   setStatus: function (newstatus) {
diff -r 743d566c0db4 -r b2d48f7ed810 web/org.openbravo.mobile.core/source/component/ob-terminal-component.js
--- a/web/org.openbravo.mobile.core/source/component/ob-terminal-component.js	Wed Apr 12 11:29:47 2017 +0200
+++ b/web/org.openbravo.mobile.core/source/component/ob-terminal-component.js	Wed Apr 12 12:42:04 2017 +0200
@@ -197,14 +197,18 @@
     });
   },
 
-  enterEvent: function (k) {
-    OB.debug('ENTER event for iOS');
-    _.each(this.keyReceivers, function (receiver) {
-      receiver.waterfall('onGlobalKeypress', {
-        keyboardEvent: k,
-        enterEvent: true
+  iosKeyEvent: function (k) {
+    OB.debug('key event for iOS - keyCode: ' + k.keyCode + " - keyIdentifier: " + k.keyIdentifier + " - charCode: " + k.charCode + " - which: " + k.which);
+    var me = this;
+    var timeoutTime = k.which === 13 ? 200 : 1;
+    setTimeout(function () {
+      _.each(me.keyReceivers, function (receiver) {
+        receiver.waterfall('onGlobalKeypress', {
+          keyboardEvent: k,
+          enterEvent: true
+        });
       });
-    });
+    }, timeoutTime);
   },
 
   /**
@@ -365,17 +369,14 @@
       // listening to keypress and then sending it to the standard flow.
       if (OB.UTIL.isIOS()) {
         keeper.addEventListener('keypress', function (k) {
-          OB.debug('timeout happened');
-          OB.MobileApp.view.enterEvent(k);
+          OB.MobileApp.view.iosKeyEvent(k);
+        });
+      } else {
+        keeper.addEventListener('keyup', function (k) {
+          OB.debug('keyup happened: which:' + k.which);
+          OB.MobileApp.view.keeperKey(k);
         });
       }
-      keeper.addEventListener('keyup', function (k) {
-        OB.debug('keyup happened: which:' + k.which);
-        if (OB.UTIL.isIOS()) {
-          return;
-        }
-        OB.MobileApp.view.keeperKey(k);
-      });
       document.body.appendChild(keeper);
     }
     this.scanningFocus();