[IMP] attempt to make handling of readonly & states.readonly slightly clearer

bzr revid: xmo@openerp.com-20110831131141-wu0f4p6etaou3xdl
2011-08-31 15:11:41 +02:00
parent 41d3a51c82
commit 5e64318f94
1 changed files with 4 additions and 5 deletions
@@ -1100,11 +1100,10 @@ class Export(View):
        records = []
        for field_name, field in fields_seq:
            if import_compat and field.get('readonly'):
-                ok = False
-                for sl in field.get('states', {}).values():
-                    for s in sl:
-                        ok = ok or (s==['readonly',False])
-                if not ok: continue
+                # If none of the field's states unsets readonly, skip the field
+                if all(dict(attrs).get('readonly', True)
+                       for attrs in field.get('states', {}).values()):
+                    continue

            id = prefix + (prefix and '/'or '') + field_name
            nm = name + (name and '/' or '') + field['string']