Rifakat (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-client/6.0-validation_O2M-rha into 
lp:openobject-client/6.0.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client/6.0-validation_O2M-rha/+merge/68650

validation on O2M field should not remove the child models

-- 
https://code.launchpad.net/~openerp-dev/openobject-client/6.0-validation_O2M-rha/+merge/68650
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-client/6.0-validation_O2M-rha.
=== modified file 'bin/widget/model/field.py'
--- bin/widget/model/field.py	2011-02-16 13:25:16 +0000
+++ bin/widget/model/field.py	2011-07-21 10:03:30 +0000
@@ -454,10 +454,7 @@
         ok = True
         for model2 in model.value[self.name].models:
             if not model2.validate():
-                if not model2.is_modified():
-                    model.value[self.name].models.remove(model2)
-                else:
-                    ok = False
+                ok = False
         if not super(O2MField, self).validate(model):
             ok = False
         self.get_state_attrs(model)['valid'] = ok

_______________________________________________
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