Mayur Maheshwari(OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/trunk-opw-579824-port-mma into 
lp:openobject-addons.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-579824-port-mma/+merge/135057

Hello,

        I have fixed the issue of unlink hr.employee, there is already 
on-delete cascade with resource.resource so whenever resource is deleted 
employee will also be deleted so no need to call further unlink of hr.employee 
as it is already unlinked with resource.
        code is forward port from 6.1

Thanks.
Mayur
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-579824-port-mma/+merge/135057
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-opw-579824-port-mma.
=== modified file 'hr/hr.py'
--- hr/hr.py	2012-11-13 15:03:37 +0000
+++ hr/hr.py	2012-11-20 06:15:25 +0000
@@ -229,7 +229,7 @@
             if resource:
                 resource_ids.append(resource.id)
         if resource_ids:
-            resource_obj.unlink(cr, uid, resource_ids, context=context)
+            return resource_obj.unlink(cr, uid, resource_ids, context=context)
         return super(hr_employee, self).unlink(cr, uid, ids, context=context)
 
     def onchange_address_id(self, cr, uid, ids, address, context=None):

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : openerp-dev-gtk@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to