From c4493dae4b5984b89a3badf140c696e8c5f85ab9 Mon Sep 17 00:00:00 2001
From: "Meera Trambadia (OpenERP)" <mtr@tinyerp.com>
Date: Tue, 17 Jul 2012 12:51:28 +0530
Subject: [PATCH] [IMP] l10n_in_hr_payroll: added boolean field on payslip run
 to indicate that 'Payment Advice' exist or not.

bzr revid: mtr@tinyerp.com-20120717072128-fc9hxxlqnrnh6l6o
---
 .../l10n_in_hr_payroll/l10n_in_hr_payroll.py  | 29 ++++++++++++++-----
 .../l10n_in_hr_payroll_view.xml               |  3 ++
 2 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/addons/l10n_in_hr_payroll/l10n_in_hr_payroll.py b/addons/l10n_in_hr_payroll/l10n_in_hr_payroll.py
index 57d7a492a97..912f7ca5c1e 100644
--- a/addons/l10n_in_hr_payroll/l10n_in_hr_payroll.py
+++ b/addons/l10n_in_hr_payroll/l10n_in_hr_payroll.py
@@ -104,7 +104,7 @@ hr_employee()
 class payroll_advice(osv.osv):
     '''
     Bank Advice
-    '''        
+    '''
     _name = 'hr.payroll.advice'
     _description = 'Bank Advice'
     _columns = {
@@ -206,13 +206,24 @@ class payroll_advice(osv.osv):
                 res.update({'bank': company.partner_id.bank_ids[0].bank.name})
         return {
             'value':res
-        }     
+        }
 payroll_advice()
 
 class hr_payslip_run(osv.osv):
 
     _inherit = 'hr.payslip.run'
     _description = 'Payslip Batches'
+    _columns = {
+        'payment_advice': fields.boolean('Payment Advice', help="If True,indicates that Payment Advice exists.", readonly=True),
+    }
+
+    _defaults = {
+        'payment_advice': False,
+    }
+
+    def draft_payslip_run(self, cr, uid, ids, context=None):
+        self.write(cr, uid, ids, {'payment_advice': False}, context=context)
+        return super(hr_payslip_run, self).draft_payslip_run(cr, uid, ids, context=context)
 
     def create_advice(self, cr, uid, ids, context=None):
         wf_service = netsvc.LocalService("workflow")
@@ -247,6 +258,10 @@ class hr_payslip_run(osv.osv):
                             'bysal': line.total
                     }
                     advice_line_pool.create(cr, uid, advice_line, context=context)
+            if run.payment_advice == False:
+                self.write(cr, uid, ids, {'payment_advice' : True})
+            else:
+                raise osv.except_osv(_('Error !'), _("Payment advice already exists for %s, 'Set to Draft' to create a new advice.") %(run.name))
         return True
 
 hr_payslip_run()
@@ -254,7 +269,7 @@ hr_payslip_run()
 class payroll_advice_line(osv.osv):
     '''
     Bank Advice Lines
-    '''    
+    '''
     def onchange_employee_id(self, cr, uid, ids, employee_id=False, context=None):
         res = {}
         hr_obj = self.pool.get('hr.employee')
@@ -262,8 +277,8 @@ class payroll_advice_line(osv.osv):
             return {'value': res}
         employee = hr_obj.browse(cr, uid, [employee_id], context=context)[0]
         res.update({'name': employee.bank_account_id.acc_number ,'ifsc_code': employee.bank_account_id.bank_bic})
-        return {'value': res}  
-        
+        return {'value': res}
+
     _name = 'hr.payroll.advice.line'
     _description = 'Bank Advice Lines'
     _columns = {
@@ -277,8 +292,8 @@ class payroll_advice_line(osv.osv):
     }
     _defaults = {
         'debit_credit': 'C',
-    }    
-    
+    }
+
 
 payroll_advice_line()
 
diff --git a/addons/l10n_in_hr_payroll/l10n_in_hr_payroll_view.xml b/addons/l10n_in_hr_payroll/l10n_in_hr_payroll_view.xml
index 84dc870e512..c2119eecefd 100644
--- a/addons/l10n_in_hr_payroll/l10n_in_hr_payroll_view.xml
+++ b/addons/l10n_in_hr_payroll/l10n_in_hr_payroll_view.xml
@@ -48,6 +48,9 @@
             <field name="inherit_id" ref="hr_payroll.hr_payslip_run_form"/>
             <field name="arch" type="xml">
             <data>
+                <xpath expr="//field[@name='credit_note']" position="after">
+                    <field name="payment_advice"/>
+                </xpath>
                  <xpath expr="//button[@string='Set to Draft']" position="after">
                     <button name="create_advice" string="Create Advice" type="object" states="close" />
                  </xpath>