diff options
| author | Xavier Morel <xmo@openerp.com> | 2012-07-18 12:17:44 +0200 |
|---|---|---|
| committer | Xavier Morel <xmo@openerp.com> | 2012-07-18 12:17:44 +0200 |
| commit | 71ede66dce1b89ce3c2c9e1ff93c1e1cae2da397 (patch) | |
| tree | 6fca27f6090a326541b0bb9574a0b705288ec9f3 /addons/web/static/src/js/view_list_editable.js | |
| parent | 01e6e3ec079c3221d14903a4a6aa066e0b53dafb (diff) | |
[FIX] handling of @onwrite
extraction was broken: using _.bind/3 will partially apply the
function (first argument) on top of binding it to a context (second
argument), but the partial application will be from the *left* (which
is pretty much the only one making sense, short of kwargs).
handle_onwrite_record took its arguments the wrong way around: the
partially applied one was specified on the right and the "actually
applied" one was on the left, so it used the wrong argument and ended
up blowing up the rest of the code.
bzr revid: xmo@openerp.com-20120718101744-bbbshq60x2kqhob6
Diffstat (limited to 'addons/web/static/src/js/view_list_editable.js')
| -rw-r--r-- | addons/web/static/src/js/view_list_editable.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/addons/web/static/src/js/view_list_editable.js b/addons/web/static/src/js/view_list_editable.js index fa39329fc81..baff7085e0e 100644 --- a/addons/web/static/src/js/view_list_editable.js +++ b/addons/web/static/src/js/view_list_editable.js @@ -352,7 +352,7 @@ openerp.web.list_editable = function (instance) { _.bind(self.handle_onwrite_record, self, source_record))); }); }, - handle_onwrite_record: function (id, source_record) { + handle_onwrite_record: function (source_record, id) { var record = this.records.get(id); if (!record) { // insert after the source record |