From bef860db74f516356dbb4f701e076b35f930f8bd Mon Sep 17 00:00:00 2001
From: "Vir (Open ERP)" <vir@tinyerp.com>
Date: Fri, 15 Oct 2010 11:31:58 +0530
Subject: [PATCH] [Fix] hr : problem of ids in job position form view

bzr revid: vir@tinyerp.com-20101015060158-yhibx8xn80m30gl5
---
 addons/hr/hr.py       | 9 ++++-----
 addons/hr/hr_view.xml | 4 ++--
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/addons/hr/hr.py b/addons/hr/hr.py
index 3e789709195..ffae26d2949 100644
--- a/addons/hr/hr.py
+++ b/addons/hr/hr.py
@@ -85,8 +85,8 @@ class hr_job(osv.osv):
     _columns = {
         'name': fields.char('Job Name', size=128, required=True, select=True),
         'expected_employees': fields.integer('Expected Employees', help='Required number of Employees'),
-        'no_of_employee': fields.integer('No of Employees', help='Number of employee there are already in the department', readonly=True),
-        'no_of_recruitment': fields.integer('No of Recruitment'),
+        'no_of_employee': fields.integer('No of Employees', help='Number of employee there are already in the department'),
+        'no_of_recruitment': fields.integer('No of Recruitment', readonly=True),
         'employee_ids': fields.one2many('hr.employee', 'job_id', 'Employees'),
         'description': fields.text('Job Description'),
         'requirements': fields.text('Requirements'),
@@ -101,13 +101,12 @@ class hr_job(osv.osv):
         'no_of_recruitment': 1,
     }
 
-    def on_change_expected_employee(self, cr, uid, ids, expected_employee, context=None):
+    def on_change_expected_employee(self, cr, uid, ids, expected_employee, no_of_employee, context=None):
         if context is None:
             context = {}
         result={}
         if expected_employee:
-            xx  = self.browse(cr, uid, ids, context)[0]
-            result['no_of_recruitment'] = expected_employee - xx['no_of_employee']
+            result['no_of_recruitment'] = expected_employee - no_of_employee
         return {'value': result}
 
     def job_old(self, cr, uid, ids, *args):
diff --git a/addons/hr/hr_view.xml b/addons/hr/hr_view.xml
index eb0601002f1..d47579a5d43 100644
--- a/addons/hr/hr_view.xml
+++ b/addons/hr/hr_view.xml
@@ -324,8 +324,8 @@
                     <field name="name" />
                     <field name="department_id" />
                     <field name="company_id" widget="selection" groups="base.group_multi_company"/>
-                    <field name="expected_employees" on_change="on_change_expected_employee(expected_employees)"/>
-                    <field name="no_of_employee"/>
+                    <field name="expected_employees" on_change="on_change_expected_employee(expected_employees,no_of_employee)"/>
+                    <field name="no_of_employee" on_change="on_change_expected_employee(expected_employees,no_of_employee)"/>
                     <field name="no_of_recruitment"/>
                     </group>
                     <newline/>