diff --git a/openerp/addons/base/ir/ir_ui_view.py b/openerp/addons/base/ir/ir_ui_view.py index d19e3e7dae2..ba5f72f0f62 100644 --- a/openerp/addons/base/ir/ir_ui_view.py +++ b/openerp/addons/base/ir/ir_ui_view.py @@ -539,7 +539,7 @@ class view(osv.osv): separator = child.get('separator', ',') if separator == ' ': separator = None # squash spaces - to_add = map(str.strip, child.get('add', '').split(separator)) + to_add = filter(bool, map(str.strip, child.get('add', '').split(separator))) to_remove = map(str.strip, child.get('remove', '').split(separator)) values = map(str.strip, node.get(attribute, '').split(separator)) value = (separator or ' ').join(filter(lambda s: s not in to_remove, values) + to_add) diff --git a/openerp/addons/base/tests/test_views.py b/openerp/addons/base/tests/test_views.py index 57a491fc20f..c0358c29189 100644 --- a/openerp/addons/base/tests/test_views.py +++ b/openerp/addons/base/tests/test_views.py @@ -746,7 +746,7 @@ class test_views(ViewCase): bob tata lolo bob, tata,lolo - + """ }) @@ -766,7 +766,7 @@ class test_views(ViewCase): E.p("Replacement data"), E.footer( E.button(name="action_next", type="object", string="New button"), - thing="bob lolo bibi and co", otherthing="lolo,bibi" + thing="bob lolo bibi and co", otherthing="lolo" ), string="Replacement title", version="7.0"))