diff --git a/addons/web/static/src/js/views/abstract_renderer.js b/addons/web/static/src/js/views/abstract_renderer.js index 2394fb31a64..059380f87bd 100644 --- a/addons/web/static/src/js/views/abstract_renderer.js +++ b/addons/web/static/src/js/views/abstract_renderer.js @@ -66,6 +66,11 @@ return Widget.extend({ */ getLocalState: function () { }, + /** + * Order to focus to be given to the content of the current view + */ + giveFocus: function () { + }, /** * This is the reverse operation from getLocalState. With this method, we * expect the renderer to restore all DOM state, if it is relevant. diff --git a/addons/web/static/src/js/views/basic/basic_renderer.js b/addons/web/static/src/js/views/basic/basic_renderer.js index 80cdd05bff5..1dae9f705ff 100644 --- a/addons/web/static/src/js/views/basic/basic_renderer.js +++ b/addons/web/static/src/js/views/basic/basic_renderer.js @@ -135,13 +135,6 @@ var BasicRenderer = AbstractRenderer.extend({ dom.setSelectionRange(field.getFocusableElement().get(0), {start: offset, end: offset}); } }, - - /** - * Order to focus to be given to the content of the current view - */ - giveFocus:function() { - }, - //-------------------------------------------------------------------------- // Private //-------------------------------------------------------------------------- diff --git a/addons/web/static/tests/views/graph_tests.js b/addons/web/static/tests/views/graph_tests.js index a13130e4b27..028eb4cb3a3 100644 --- a/addons/web/static/tests/views/graph_tests.js +++ b/addons/web/static/tests/views/graph_tests.js @@ -602,6 +602,22 @@ QUnit.module('Views', { done(); }); }); + + QUnit.test('graph view crash when moving from search view using Down key', function (assert) { + assert.expect(1); + + var graph = createView({ + View: GraphView, + model: "foo", + data: this.data, + arch: '' + + '' + + '', + }); + graph.renderer.giveFocus(); + assert.ok(true,"should not generate any error"); + graph.destroy(); + }); }); });