From c83acd9ce4e04cadac352fa25db4e42c2a235f0e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?St=C3=A9phane=20Bidoul?= <>
Date: Wed, 15 Jun 2011 15:34:11 +0530
Subject: [PATCH] [ADD] hr : Added default value for departmet and applied
 onchange method for department according to employee_id

lp bug: https://launchpad.net/bugs/708730 fixed

bzr revid: bde@tiny2-desktop-20110615100411-gs5vkug3kb0i8sg8
---
 addons/hr_timesheet_sheet/hr_timesheet_sheet.py   | 15 ++++++++++++++-
 .../hr_timesheet_sheet_view.xml                   |  2 +-
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/addons/hr_timesheet_sheet/hr_timesheet_sheet.py b/addons/hr_timesheet_sheet/hr_timesheet_sheet.py
index ea2b5017790..94b390b30a2 100644
--- a/addons/hr_timesheet_sheet/hr_timesheet_sheet.py
+++ b/addons/hr_timesheet_sheet/hr_timesheet_sheet.py
@@ -321,9 +321,15 @@ class hr_timesheet_sheet(osv.osv):
             return time.strftime('%Y-12-31')
         return time.strftime('%Y-%m-%d')
 
-    def _default_employee(self,cr, uid, context=None):
+    def _default_employee(self, cr, uid, context=None):
         emp_ids = self.pool.get('hr.employee').search(cr, uid, [('user_id','=',uid)], context=context)
         return emp_ids and emp_ids[0] or False
+    
+    def _default_department(self, cr, uid, context=None):
+        emp_id = self._default_employee(cr, uid, context)
+        if emp_id:
+            return self.pool.get('hr.employee').read(cr, uid, [emp_id], context)[0]['department_id']
+        return False
 
     _defaults = {
         'date_from' : _default_date_from,
@@ -331,6 +337,7 @@ class hr_timesheet_sheet(osv.osv):
         'date_to' : _default_date_to,
         'state': 'new',
         'employee_id': _default_employee,
+        'department_id': _default_department,
         'company_id': lambda self, cr, uid, c: self.pool.get('res.company')._company_default_get(cr, uid, 'hr_timesheet_sheet.sheet', context=c)
     }
 
@@ -382,6 +389,12 @@ class hr_timesheet_sheet(osv.osv):
                 raise osv.except_osv(_('Invalid action !'), _('Cannot delete Sheet(s) which have attendance entries encoded !'))
         return super(hr_timesheet_sheet, self).unlink(cr, uid, ids, context=context)
 
+    def onchange_department_id(self, cr, uid, ids, employee_id, context=None):
+        department_id = False
+        if employee_id:
+            department_id = self.pool.get('hr.employee').browse(cr, uid, employee_id, context=context).department_id.id or False
+        return {'value': {'department_id': department_id}}
+
 hr_timesheet_sheet()
 
 
diff --git a/addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml b/addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml
index 321e4c2addd..dc43ca10920 100644
--- a/addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml
+++ b/addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml
@@ -66,7 +66,7 @@
                         <field name="company_id" select="1" groups="base.group_multi_company"/>
                         <field name="department_id" groups="base.group_extended"/>
                         <newline/>
-                        <field name="employee_id"/>
+                        <field name="employee_id" on_change="onchange_department_id(employee_id)"/>
                         <field name="user_id" invisible="1"/>
                         <field name="date_from"/>
                         <field name="date_to"/>