openerp-dev-web team mailing list archive

["Ashvin Rathod \(OpenERP\)" <ara@xxxxxxxxxxx>]

Ashvin Rathod (OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/trunk-bug-769632-ara into lp:openobject-addons.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #769632 in OpenERP Addons: "Removing Administrator employee record causes future updates of hr module to fail"
  https://bugs.launchpad.net/openobject-addons/+bug/769632

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-769632-ara/+merge/59044

Hello,

Fix: HR-Removing Administrator employee record causes future updates of hr module to fail

Thanks,
ara
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-769632-ara/+merge/59044
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-769632-ara.

=== modified file 'hr/hr.py'
--- hr/hr.py	2011-04-07 09:27:30 +0000
+++ hr/hr.py	2011-04-26 11:31:05 +0000
@@ -161,6 +161,17 @@
         'passport_id':fields.char('Passport No', size=64)
     }
 
+    def unlink(self, cr, uid, ids, context=None):
+        resource_obj = self.pool.get('resource.resource')
+        resource_ids = []
+        for employee in self.browse(cr, uid, ids, context=context):
+            resource = employee.resource_id
+            if resource:
+                resource_ids.append(resource.id)
+        if resource_ids:
+            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):
         if address:
             address = self.pool.get('res.partner.address').browse(cr, uid, address, context=context)