openerp-dev-web team mailing list archive

["Jiten \(OpenERP\)" <jra@xxxxxxxxxxx>]

Jiten (OpenERP) has proposed merging lp:~openerp-dev/openobject-client-web/trunk-bug-720173-jra into lp:openobject-client-web.

Requested reviews:
  OpenERP SA's Web Client R&D (openerp-dev-web)
Related bugs:
  Bug #720173 in OpenERP Web Client: "O2Ms nested in O2M form can not be used if the O2M form object has no mandatory attribute"
  https://bugs.launchpad.net/openobject-client-web/+bug/720173

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/trunk-bug-720173-jra/+merge/54861

In case of O2Ms nested in O2M form if the parent id is not saved/created than it should not allow to create new record.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/trunk-bug-720173-jra/+merge/54861
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/trunk-bug-720173-jra.

=== modified file 'addons/openerp/widgets/form/_o2m.py'
--- addons/openerp/widgets/form/_o2m.py	2011-02-10 16:09:23 +0000
+++ addons/openerp/widgets/form/_o2m.py	2011-03-25 13:32:31 +0000
@@ -210,6 +210,12 @@
         if 'default_name' in current.context:
             del current.context['default_name']
 
+        if self.view_type == 'tree' and pparams:
+            if not pparams.id:
+                self.editable = False
+            else:
+                self.editable = True
+
         self.screen = Screen(current, prefix=self.name, views_preloaded=view,
                              editable=self.editable, readonly=self.readonly,
                              selectable=0, nolinks=self.link, _o2m=1)