[IMP] small change in holidays_confirm method so it will raise warning for only leave requests

bzr revid: psi@tinyerp.co.in-20110331123742-gc0vfk5a1d8gq10w
This commit is contained in:
psi (Open ERP)
2011-03-31 18:07:42 +05:30
parent e9b4d4538c
commit d9f0d9f279
+2 -2
View File
@@ -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()