From 2d08b98ea4735dfaa54eac2ffd6c6b780bb02e66 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Thu, 9 Feb 2012 16:48:44 +0100 Subject: [PATCH] [FIX] bug introduced during xmo@openerp.com-20120208130738-pr92lh9qotki71o4 Change made registry inheritance "lazy", keeping a permanent relation between child and parent and walking the tree to find matches. But editable listviews relied on an implementation detail (that an extension eagerly created keys in the children registry) and iterated over the internal map of the child registry they'd just created to replace the form keys with overridden versions of the form widgets. Just iterate over form's registry, but really the editable list should have its own Registry subclass doing that stuff more solidly. bzr revid: xmo@openerp.com-20120209154844-zk2xuxdhhi57eqhp --- addons/web/static/src/js/view_list_editable.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/web/static/src/js/view_list_editable.js b/addons/web/static/src/js/view_list_editable.js index 3eae086539b..c7222455889 100644 --- a/addons/web/static/src/js/view_list_editable.js +++ b/addons/web/static/src/js/view_list_editable.js @@ -395,7 +395,7 @@ openerp.web.list_editable = function (openerp) { // former should completely remove the cell. We need to override update_dom // on all widgets since we can't just hit on widget itself (I think) var list_form_widgets = openerp.web.list.form.widgets; - _(list_form_widgets.map).each(function (widget_path, key) { + _(form_widgets.map).each(function (widget_path, key) { if (key === 'frame') { return; } var new_path = 'openerp.web.list.form.' + key;