clearcorp team mailing list archive

[Ronald Rubi <ronald.rubi@xxxxxxxxxxxxxxx>]

Ronald Rubi has proposed merging lp:~rr.clearcorp/openerp-costa-rica/6.1-l10n_cr_hr_payroll into lp:openerp-costa-rica/6.1.

Requested reviews:
  CLEARCORP drivers (clearcorp-drivers)

For more details, see:
https://code.launchpad.net/~rr.clearcorp/openerp-costa-rica/6.1-l10n_cr_hr_payroll/+merge/164975

[FIX] Fix confirm_payslips in batches
-- 
https://code.launchpad.net/~rr.clearcorp/openerp-costa-rica/6.1-l10n_cr_hr_payroll/+merge/164975
Your team CLEARCORP development team is subscribed to branch lp:openerp-costa-rica/6.1.

=== modified file 'l10n_cr_hr_payroll/l10n_cr_hr_payroll.py'
--- l10n_cr_hr_payroll/l10n_cr_hr_payroll.py	2013-03-22 17:43:43 +0000
+++ l10n_cr_hr_payroll/l10n_cr_hr_payroll.py	2013-05-21 20:21:32 +0000
@@ -82,14 +82,14 @@
     }
 
     def confirm_payslips(self, cr, uid, ids, context=None):
-        for payslip_run in self.browse(cr, uid, ids, context=context):
-            payslip_obj = self.pool.get('hr.payslip')
-            payslips = payslip_obj.browse(cr, uid, payslip_run.slip_ids, context)
-            for payslip in payslips:
-                payslip_id = payslip.id
-                payslip_id.compute_sheet()
-                payslip_id.process_sheet()
-
+        payslip_obj = self.pool.get('hr.payslip')
+        for batches in self.browse(cr, uid, ids, context=context):
+            payslip_ids = map(lambda x: x.id, batches.slip_ids)
+            for payslip in payslip_obj.browse(cr, uid, payslip_ids):
+                    if payslip.state == 'draft':
+                        payslip_obj.process_sheet(cr, uid, [payslip.id], context=context)
+        return True
+        
 hr_payslip_run()
 
 class HrPayslip(osv.osv):