From 9d09c570aefc8c96a026a4316937c7a42ff6cf3d Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Mon, 21 Nov 2011 13:28:45 +0100 Subject: [PATCH] [ADD] support for arguments-less onchange (why the hell not) bzr revid: xmo@openerp.com-20111121122845-98lctm470ncttz74 --- addons/web/static/src/js/view_form.js | 7 +++++-- addons/web/static/test/onchange.js | 12 ++++++------ 2 files changed, 11 insertions(+), 8 deletions(-) diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js index 1bb43a0e4b8..1f35c0e1dc3 100644 --- a/addons/web/static/src/js/view_form.js +++ b/addons/web/static/src/js/view_form.js @@ -250,8 +250,11 @@ openerp.web.FormView = openerp.web.View.extend( /** @lends openerp.web.FormView# return null; } - var method = call[1], args = []; - var context_index = null; + var method = call[1], args = [], context_index = null; + if (!_.str.trim(call[2])) { + return {method: method, args: args, context_index: context_index} + } + var argument_replacement = { 'False': function () {return false;}, 'True': function () {return true;}, diff --git a/addons/web/static/test/onchange.js b/addons/web/static/test/onchange.js index 07eb6c1bc2c..c606915f31a 100644 --- a/addons/web/static/test/onchange.js +++ b/addons/web/static/test/onchange.js @@ -22,12 +22,12 @@ $(document).ready(function () { window.openerp.web.form(openerp); } }); -// test('Parse args-less onchange', function () { -// var f = new openerp.web.FormView(null, {}); -// var result = f.parse_on_change('on_change_foo()', {}); -// equal(result.method, 'on_change_foo'); -// deepEqual(result.args, []); -// }); + test('Parse args-less onchange', function () { + var f = new openerp.web.FormView(null, {}); + var result = f.parse_on_change('on_change_foo()', {}); + equal(result.method, 'on_change_foo'); + deepEqual(result.args, []); + }); test('Parse 1-arg onchange', function () { var f = make_form({foo: 3}); var result = f.parse_on_change('on_change_foo(foo)', {});