Review: Needs Fixing

This patch looks very wrong, now the "if resource_ids" test will never pass as 
'resource_ids' is initialized to an empty list in the previous line. Please 
review your patches before submitting such nonsense ;-)
If you'd like to delete the employees before the resources they inherit from 
you should simply put the call to resource.unlink() *after* the call to 
super().unlink().
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.1-opw-578933-nep/+merge/123733
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.1-opw-578933-nep.

_______________________________________________
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