From 4de1ee3f357fb4dbfb42a27cc2c0fc561da810b7 Mon Sep 17 00:00:00 2001
From: "psi (Open ERP)" <psi@tinyerp.co.in>
Date: Wed, 9 Mar 2011 15:40:41 +0530
Subject: [PATCH] [IMP] on hr.employee object make basic field as function and
 the function make the sum of basic field of all the hr.contract for this
 employee that have date_start < now() and now() < date_stop or not date_stop

bzr revid: psi@tinyerp.co.in-20110309101041-p4i0n49uq0qo2tm2
---
 addons/hr_payroll/hr_payroll.py       | 25 +++++++++++++++++++++----
 addons/hr_payroll/hr_payroll_view.xml |  1 +
 2 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/addons/hr_payroll/hr_payroll.py b/addons/hr_payroll/hr_payroll.py
index 08e15045de7..8a45b916a37 100644
--- a/addons/hr_payroll/hr_payroll.py
+++ b/addons/hr_payroll/hr_payroll.py
@@ -96,7 +96,6 @@ class hr_contract(osv.osv):
     _columns = {
         'struct_id': fields.many2one('hr.payroll.structure', 'Salary Structure'),
         'basic': fields.float('Basic Salary', digits_compute=dp.get_precision('Account')), # i think we can remove this because we have wage field on contract ?
-        #FIXME not needed as we have the resource.calendar?        'working_days_per_week': fields.integer('Working Days', help="No of Working days / week for an employee"),
     }
 
 hr_contract()
@@ -1249,16 +1248,34 @@ class hr_employee(osv.osv):
     Employee
     '''
 
+    def _calculate_basic(self, cr, uid, ids, name, args, context):
+        if not ids: return {}
+        vals = {}
+        current_date = datetime.now().strftime('%Y-%m-%d')
+        for employee in self.browse(cr, uid, ids, context=context):
+            if not employee.contract_id:
+                vals[employee.id] = {'basic': 0.0}
+                continue
+            cr.execute( 'SELECT SUM(wage) '\
+                        'FROM hr_contract '\
+                        'WHERE employee_id = %s '\
+                        'AND date_start < %s '\
+                        'AND (date_end > %s OR date_end is NULL)',
+                         (employee.id, current_date, current_date))
+            result = dict(cr.dictfetchone())
+            vals[employee.id] = {
+                                 'basic': result['sum']
+            }
+        return vals
+
     _inherit = 'hr.employee'
     _description = 'Employee'
 
     _columns = {
         'line_ids':fields.one2many('hr.payslip.line', 'employee_id', 'Salary Structure', required=False),
         'slip_ids':fields.one2many('hr.payslip', 'employee_id', 'Payslips', required=False, readonly=True),
-        'otherid': fields.char('Other Id', size=64),
-#        'basic': fields.function(_calculate_salary, method=True, multi='dc', type='float', string='Basic Salary', digits=(14,2)),
+        'basic': fields.function(_calculate_basic, method=True, multi='dc', type='float', string='Basic Salary', digits_compute=dp.get_precision('Account')),
 #FIXME the function has to make the sum of basic field of all the hr.contract for this employee that have date_start < now() and now() < date_stop or not date_stop
-#        'emp_sal_rule_ids':fields.many2many('hr.salary.rule', 'hr_emp_salary_rule_rel', 'employee_id', 'rule_id', 'Salary Rules', readonly=False),
     }
 hr_employee()
 
diff --git a/addons/hr_payroll/hr_payroll_view.xml b/addons/hr_payroll/hr_payroll_view.xml
index ea6e1ca7dd3..a2f3ac5b859 100644
--- a/addons/hr_payroll/hr_payroll_view.xml
+++ b/addons/hr_payroll/hr_payroll_view.xml
@@ -20,6 +20,7 @@
                             <field name="bank_account_id" context="{'display_partner':True, 'partner_id':partner_id}"/>
                             <field name="place_of_birth"/>
                             <field name="children"/>
+                            <field name="basic" invisible="1"/>
                         </group>
                         <group colspan="2" col="2">
                             <separator string="Job Info" colspan="2"/>