From f892e30a6f395162229c8eef54b62b550bc8f0cb Mon Sep 17 00:00:00 2001
From: Jairo Llopis <yajo.sk8@gmail.com>
Date: Thu, 7 Sep 2017 15:10:41 +0200
Subject: [PATCH] [FIX][IMP][web_decimal_numpad_dot] Support float time

- All Char widgets were getting affected by this behavior.
- Time float widgets now get ":" instead of "," or ".".
- Little code refactor.
- Only detect numpad dot, not normal dot and comma.
---
 web_decimal_numpad_dot/README.rst             |  1 +
 web_decimal_numpad_dot/__manifest__.py        |  2 +-
 .../static/src/js/numpad_dot.js               | 51 +++++++++++--------
 3 files changed, 32 insertions(+), 22 deletions(-)

diff --git a/web_decimal_numpad_dot/README.rst b/web_decimal_numpad_dot/README.rst
index 71f5e51c..fd318468 100644
--- a/web_decimal_numpad_dot/README.rst
+++ b/web_decimal_numpad_dot/README.rst
@@ -40,6 +40,7 @@ Contributors
 * Oliver Dony <@odony>
 * Wim Audenaert <Wim.Audenaert@ucamco.com>
 * David Vidal <david.vidal@tecnativa.com>
+* Jairo Llopis <jairo.llopis@tecnativa.com>
 
 Maintainer
 ----------
diff --git a/web_decimal_numpad_dot/__manifest__.py b/web_decimal_numpad_dot/__manifest__.py
index 457c130b..10c957dc 100644
--- a/web_decimal_numpad_dot/__manifest__.py
+++ b/web_decimal_numpad_dot/__manifest__.py
@@ -8,7 +8,7 @@
 
 {
     "name": "Web - Numpad Dot as decimal separator",
-    "version": "10.0.1.0.0",
+    "version": "10.0.1.1.0",
     "license": "AGPL-3",
     "summary": "Allows using numpad dot to enter period decimal separator",
     "depends": [
diff --git a/web_decimal_numpad_dot/static/src/js/numpad_dot.js b/web_decimal_numpad_dot/static/src/js/numpad_dot.js
index 8d227803..4e8b7de4 100644
--- a/web_decimal_numpad_dot/static/src/js/numpad_dot.js
+++ b/web_decimal_numpad_dot/static/src/js/numpad_dot.js
@@ -8,27 +8,36 @@ odoo.define("web_decimal_numpad_dot.FieldFloat", function (require) {
 
     form_widgets.FieldFloat.include({
         init: function () {
-            this.events.keypress = function (event) {
-                if (event.which === 46 || event.which === 44) {
-                    event.preventDefault();
-                    var input = this.$input || this.$("input");
-                    var l10n = translation._t.database.parameters;
-                    if (!_.str.contains(input.val(), l10n.decimal_point)) {
-                        try {
-                            var caret_pos = input[0].selectionStart;
-                            var selection_end = input[0].selectionEnd;
-                            var cur_val = input.val();
-                            var newval = cur_val.substring(0, caret_pos) + l10n.decimal_point + cur_val.substring(selection_end);
-                            input.val(newval);
-                            input[0].selectionStart = input[0].selectionEnd = caret_pos + 1;
-                        } catch (error) {
-                            //fallback to appending if no caret position can be determined
-                            input.val(input.val() + l10n.decimal_point);
-                        }
-                    }
-                }
-            };
+            this.events = $.extend({}, this.events, {
+                "keydown": "numpad_dot_replace",
+            });
             return this._super.apply(this, arguments);
-        }
+        },
+
+        l10n_decimal_point: function () {
+            return this.widget == "float_time"
+                ? ":" : translation._t.database.parameters.decimal_point;
+        },
+
+        numpad_dot_replace: function (event) {
+            // Only act on numpad dot key
+            if (event.keyCode != 110) {
+                return;
+            }
+            event.preventDefault();
+            var from = this.$input.prop("selectionStart"),
+                to = this.$input.prop("selectionEnd"),
+                cur_val = this.$input.val(),
+                point = this.l10n_decimal_point();
+            // Replace selected text by proper character
+            this.$input.val(
+                cur_val.substring(0, from) +
+                point +
+                cur_val.substring(to)
+            );
+            // Put user caret in place
+            to = from + point.length
+            this.$input.prop("selectionStart", to).prop("selectionEnd", to);
+        },
     });
 });