[FIX] bug introduced during xmo@openerp.com-20120208130738-pr92lh9qotki71o4
authorXavier Morel <xmo@openerp.com>
Thu, 9 Feb 2012 15:48:44 +0000 (16:48 +0100)
committerXavier Morel <xmo@openerp.com>
Thu, 9 Feb 2012 15:48:44 +0000 (16:48 +0100)
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

index 3eae086..c722245 100644 (file)
@@ -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;