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-731795-ara into lp:openobject-addons.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #731795 in OpenERP Addons: "[trunk] hr_holidays: Can not refuse leave if its category is "by employee category""
  https://bugs.launchpad.net/openobject-addons/+bug/731795

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

Hello,

Fix: [trunk] hr_holidays: Can not refuse leave if its category is "by employee category"
https://bugs.launchpad.net/openobject-addons/+bug/731795

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

=== modified file 'hr_holidays/hr_holidays.py'
--- hr_holidays/hr_holidays.py	2011-02-01 12:40:38 +0000
+++ hr_holidays/hr_holidays.py	2011-03-10 10:00:29 +0000
@@ -294,8 +294,8 @@
 
             # If a category that created several holidays, cancel all related
             wf_service = netsvc.LocalService("workflow")
-            for id in record.linked_request_ids or []:
-                wf_service.trg_validate(uid, 'hr.holidays', id, 'cancel', cr)
+            for request in record.linked_request_ids or []:
+                wf_service.trg_validate(uid, 'hr.holidays', request.id, 'cancel', cr)
 
         return True