From d9f0d9f279d949c2ee92e56a2efd97b16a5f7f5a Mon Sep 17 00:00:00 2001 From: "psi (Open ERP)" Date: Thu, 31 Mar 2011 18:07:42 +0530 Subject: [PATCH] [IMP] small change in holidays_confirm method so it will raise warning for only leave requests bzr revid: psi@tinyerp.co.in-20110331123742-gc0vfk5a1d8gq10w --- addons/hr_payroll/hr_payroll.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/hr_payroll/hr_payroll.py b/addons/hr_payroll/hr_payroll.py index 7a9b3160ce6..87f12da036b 100644 --- a/addons/hr_payroll/hr_payroll.py +++ b/addons/hr_payroll/hr_payroll.py @@ -1352,9 +1352,9 @@ class hr_holidays(osv.osv): def holidays_confirm(self, cr, uid, ids, *args): leaves = self.browse(cr, uid, ids) for leave in leaves: - if not leave.contract_id: + if not leave.contract_id and leave.type == 'remove': raise osv.except_osv(_('Warning !'), _("Please define contract on %s's leave request ! ") % (leave.employee_id.name)) - return super(hr_holidays, self).holidays_confirm(cr, uid, ids) + return super(hr_holidays, self).holidays_confirm(cr, uid, ids, args) hr_holidays()