[FIX] web: typo in function name (FieldBoolean._onKeydown)

When focusing a checkbox field and hitting the "enter" key, the
checkbox should change its status. Before this commit, this produced
a traceback instead as it was trying to call the `_onKeyDown`
function while it was defined as `_onKeydown`.

Also the `_onKeydown` method is in fact declared in the `AbstractField`
parent implementation, so there is no point in declaring the event
binding twice. Also, part of the `FieldBoolean._onKeydown` code was in
common with its parent `AbstractField._onKeydown` code so remove it as
there is no point in duplicating it.
Note: this `_onKeydown` method has to be refactored as it is breaking
form view navigation anyway.

Also modify an existing test to check the checkbox keyboard navigation.

1 files changed, 1 insertions, 5 deletions

diff --git a/addons/web/static/src/js/fields/basic_fields.js b/addons/web/static/src/js/fields/basic_fields.js
index 3a2aa23e0e0..245643ece57 100644
--- a/addons/web/static/src/js/fields/basic_fields.js
+++ b/addons/web/static/src/js/fields/basic_fields.js
@@ -373,7 +373,6 @@ var FieldBoolean = AbstractField.extend({
     className: 'o_field_boolean',
     events: _.extend({}, AbstractField.prototype.events, {
         change: '_onChange',
-        keydown: '_onKeyDown',
     }),
     replace_element: true,
     supportedFieldTypes: ['boolean'],
@@ -455,6 +454,7 @@ var FieldBoolean = AbstractField.extend({
      * @param {KeyEvent} event
      */
     _onKeydown: function (event) {
+        this._super.apply(this, arguments);
         switch (event.which) {
             case $.ui.keyCode.DOWN:
                 this.trigger_up('move_down');
@@ -468,10 +468,6 @@ var FieldBoolean = AbstractField.extend({
             case $.ui.keyCode.RIGHT:
                 this.trigger_up('move_right');
                 break;
-            case $.ui.keyCode.TAB:
-                this.trigger_up('move_next');
-                event.preventDefault();
-                break;
             case $.ui.keyCode.ENTER:
                 this.$input.prop('checked', !this.value);
                 this._setValue(!this.value);