Merge lp:~openerp/openobject-server/trunk-wip-niv into lp:openobject-server

[Nicolas Vanhoren (OpenERP)]

Improved modifiers.

[Vo Minh Thu (thu)]

Before merging, can you add test cases to the modifiers_tests() function?

I think you can reuse the modifiers variable instead of using a default_values intermediary variable. This would simplify the last loop a bit.

Some variable name consistency would be nice too. exceptions instead of modifs, (a, value) instead of (modif[0], modif[1]) and renaming a into attr, or attr in a.

  default_values[a] = False
    if field.get(a):
       default_values[a] = bool(field.get(a))

can be simplified to

  default_values[a] = bool(field.get(a))

  field.get("states") or {}

can be just

  field.get("states", {})

At line 18, no need for parens.

review: Approve

[Vo Minh Thu (thu)]

The approve above is a mistake. Or is modulo the changes...

[Nicolas Vanhoren (OpenERP) (niv-openerp)]

Tu sais, c'aurait été genre trois fois plus rapide que tu le fasses toi même plutôt que de m'emmerder avec de vaines critiques subjectives sur le nommage de variables.

Je te rappelle que c'est quand même ton boulot que j'ai fait à ta place parce que je voulais que ce soit plus vite fait. D'autant plus que tu n'aurais pas testé le résultat avec des cas réels, ce qui fait qu'à la moindre erreur j'aurais du revenir te demander de le fixer, puis à l'erreur suivante ç'aurait été pareil, etc...

Alors voila ce qu'on va faire: T'es pas content? Fais le toi même, moi je touche plus à rien.

[lp:~openerp/openobject-server/trunk-wip-niv]

3500. By Nicolas Vanhoren (OpenERP) on 2011-07-08

[merge]

3501. By Antony Lesuisse (OpenERP) on 2011-07-12

[FIX] apply some merge review feeback

=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py	2011-07-05 12:22:22 +0000
+++ openerp/osv/orm.py	2011-07-12 14:37:01 +0000
@@ -77,9 +77,21 @@
 ROOT_USER_ID = 1
 
 def transfer_field_to_modifiers(field, modifiers):
-    for a in ('invisible', 'readonly', 'required'):
-        if field.get(a):
-            modifiers[a] = bool(field.get(a))
+    default_values = {}
+    state_exceptions = {}
+    for attr in ('invisible', 'readonly', 'required'):
+        state_exceptions[attr] = []
+        default_values[attr] = bool(field.get(attr))
+    for state, modifs in (field.get("states",{})).items():
+        for modif in modifs:
+            if default_values[modif[0]] != modif[1]:
+                state_exceptions[modif[0]].append(state)
+
+    for attr, default_value in default_values.items():
+        if state_exceptions[attr]:
+            modifiers[attr] = [("state", "not in" if default_value else "in", state_exceptions[attr])]
+        else:
+            modifiers[attr] = default_value
 
 
 # Don't deal with groups, it is done by check_group().