[FIX] mass_editing: modifiers attribute must be valid JSON

Replace tuples with arrays in the value of the modifiers attribute.
OCA · Mar 21, 2017 · 0ef5be8 · 0ef5be8
1 parent c08d17c
commit 0ef5be8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/mass_editing/__manifest__.py b/mass_editing/__manifest__.py
@@ -4,7 +4,7 @@
 
 {
     'name': 'Mass Editing',
-    'version': '10.0.1.1.0',
+    'version': '10.0.1.1.1',
     'author': 'Serpent Consulting Services Pvt. Ltd., '
               'MONK Software, '
               'Odoo Community Association (OCA)',

diff --git a/mass_editing/wizard/mass_editing_wizard.py b/mass_editing/wizard/mass_editing_wizard.py
@@ -125,8 +125,8 @@ def fields_view_get(self, view_id=None, view_type='form', toolbar=False,
                         'colspan': '4',
                         'attrs': ("{'invisible':[('selection__" +
                                   field.name + "', '=', 'remove')]}"),
-                        'modifiers': ('{"invisible":[("selection__' +
-                                      field.name + '", "=", "remove")]}'),
+                        'modifiers': ('{"invisible":[["selection__' +
+                                      field.name + '", "=", "remove"]]}'),
                     })
                 elif field.ttype == "char":
                     all_fields["selection__" + field.name] = {
@@ -148,8 +148,8 @@ def fields_view_get(self, view_id=None, view_type='form', toolbar=False,
                         'nolabel': '1',
                         'attrs': ("{'invisible':[('selection__" +
                                   field.name + "','=','remove')]}"),
-                        'modifiers': ('{"invisible":[("selection__' +
-                                      field.name + '","=","remove")]}'),
+                        'modifiers': ('{"invisible":[["selection__' +
+                                      field.name + '","=","remove"]]}'),
                         'colspan': '4',
                     })
                 elif field.ttype == 'selection':
@@ -168,8 +168,8 @@ def fields_view_get(self, view_id=None, view_type='form', toolbar=False,
                         'colspan': '4',
                         'attrs': ("{'invisible':[('selection__" +
                                   field.name + "', '=', 'remove')]}"),
-                        'modifiers': ('{"invisible":[("selection__' +
-                                      field.name + '", "=", "remove")]}'),
+                        'modifiers': ('{"invisible":[["selection__' +
+                                      field.name + '", "=", "remove"]]}'),
                     })
                     all_fields[field.name] = {
                         'type': field.ttype,
@@ -206,8 +206,8 @@ def fields_view_get(self, view_id=None, view_type='form', toolbar=False,
                             'nolabel': '1',
                             'attrs': ("{'invisible':[('selection__" +
                                       field.name + "','=','remove')]}"),
-                            'modifiers': ('{"invisible":[("selection__' +
-                                          field.name + '","=","remove")]}'),
+                            'modifiers': ('{"invisible":[["selection__' +
+                                          field.name + '","=","remove"]]}'),
                         })
                     else:
                         all_fields["selection__" + field.name] = {
@@ -224,8 +224,8 @@ def fields_view_get(self, view_id=None, view_type='form', toolbar=False,
                             'nolabel': '1',
                             'attrs': ("{'invisible':[('selection__" +
                                       field.name + "','=','remove')]}"),
-                            'modifiers': ('{"invisible":[("selection__' +
-                                          field.name + '","=","remove")]}'),
+                            'modifiers': ('{"invisible":[["selection__' +
+                                          field.name + '","=","remove"]]}'),
                             'colspan': '4',
                         })
             etree.SubElement(xml_form, 'separator', {