Naresh(OpenERP) has proposed merging 
lp:~openerp-dev/openobject-server/trunk-bug-891580-nch into 
lp:openobject-server.

Requested reviews:
  OpenERP R&D Team (openerp-dev)
  Vo Minh Thu (OpenERP) (vmt-openerp)
  Olivier Dony (OpenERP) (odo-openerp)
Related bugs:
  Bug #891580 in OpenERP Server: "Properties should be deleted when the related 
record is deleted"
  https://bugs.launchpad.net/openobject-server/+bug/891580

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-891580-nch/+merge/84237
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-891580-nch/+merge/84237
Your team OpenERP R&D Team is requested to review the proposed merge of 
lp:~openerp-dev/openobject-server/trunk-bug-891580-nch into 
lp:openobject-server.
=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py	2011-12-01 12:15:03 +0000
+++ openerp/osv/orm.py	2011-12-02 10:29:24 +0000
@@ -3659,6 +3659,13 @@
         self.check_unlink(cr, uid)
 
         properties = self.pool.get('ir.property')
+        ##search if there are properties created with resource that is about to be deleted
+        ## if found delete the properties too.
+        unlink_properties = properties.search(cr, uid, [('res_id', 'in', ['%s,%s' % (self._name, i) for i in ids])], context=context)
+        if unlink_properties:
+            properties.unlink(cr, uid, unlink_properties, context=context)
+        
+        ## check if the model is used as a default property
         domain = [('res_id', '=', False),
                   ('value_reference', 'in', ['%s,%s' % (self._name, i) for i in ids]),
                  ]

_______________________________________________
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