Ravi Gadhia (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-client/trunk-bug-790141-rga into 
lp:openobject-client.

Requested reviews:
  Naresh(OpenERP) (nch-openerp)
Related bugs:
  Bug #790141 in OpenERP GTK Client: "Even default value presence , unable to 
save record"
  https://bugs.launchpad.net/openobject-client/+bug/790141

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client/trunk-bug-790141-rga/+merge/67134
-- 
https://code.launchpad.net/~openerp-dev/openobject-client/trunk-bug-790141-rga/+merge/67134
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-client/trunk-bug-790141-rga.
=== modified file 'bin/widget/model/group.py'
--- bin/widget/model/group.py	2011-04-13 05:28:58 +0000
+++ bin/widget/model/group.py	2011-07-07 09:04:59 +0000
@@ -405,12 +405,14 @@
             if self.parent and self.screen:
                 ctx.update(self.parent.expr_eval(self.screen.default_get))
             ctx.update(self.context)
-            values = self.rpc.default_get(to_add, ctx)
+            defalut_value = values = self.rpc.default_get(to_add, ctx)
             for t in to_add:
-                if t not in values:
-                    values[t] = False
+                if t not in defalut_value:
+                    defalut_value[t] = False
             for mod in new:
-                mod.set_default(values)
+                mod.set_default(defalut_value)
+                if values:
+                    mod.modified = True
 
     def __iter__(self):
         return iter(self.models)

=== modified file 'bin/widget/model/record.py'
--- bin/widget/model/record.py	2011-05-18 11:20:33 +0000
+++ bin/widget/model/record.py	2011-07-07 09:04:59 +0000
@@ -161,6 +161,8 @@
     def default_get(self, domain=[], context={}):
         if len(self.mgroup.fields):
             val = self.rpc.default_get(self.mgroup.fields.keys(), context)
+            if self.parent and val:
+                self.modified = True
             for d in domain:
                 if d[0] in self.mgroup.fields and not self.mgroup.fields.get(d[0], {}).get('readonly',False):
                     if d[1] == '=':

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to