[MERGE] [FIX] hr: when duplicating an employee or user, do not duplicate the referenc...
authorMartin Trigaux <mat@openerp.com>
Mon, 5 May 2014 10:13:13 +0000 (12:13 +0200)
committerMartin Trigaux <mat@openerp.com>
Mon, 5 May 2014 10:13:13 +0000 (12:13 +0200)
bzr revid: mat@openerp.com-20140505101313-5y72gs2xpgokhxl1

1  2 
addons/hr/hr.py

diff --cc addons/hr/hr.py
@@@ -211,7 -211,14 +211,13 @@@ class hr_employee(osv.osv)
      }
  
      _order='name_related'
+     
 -    def copy(self, cr, uid, ids, default=None, context=None):
++    def copy_data(self, cr, uid, ids, default=None, context=None):
+         if default is None:
+             default = {}
 -        default = default.copy()
+         default.update({'child_ids': False})
 -        return super(hr_employee, self).copy(cr, uid, ids, default, context=context)
++        return super(hr_employee, self).copy_data(cr, uid, ids, default, context=context)
+         
      def create(self, cr, uid, data, context=None):
          employee_id = super(hr_employee, self).create(cr, uid, data, context=context)
          try:
@@@ -291,17 -298,24 +297,22 @@@ class hr_department(osv.osv)
          'member_ids': fields.one2many('hr.employee', 'department_id', 'Members', readonly=True),
      }
  
--    def copy(self, cr, uid, ids, default=None, context=None):
++    def copy_data(self, cr, uid, ids, default=None, context=None):
          if default is None:
              default = {}
--        default = default.copy()
          default['member_ids'] = []
--        return super(hr_department, self).copy(cr, uid, ids, default, context=context)
++        return super(hr_department, self).copy_data(cr, uid, ids, default, context=context)
  
  class res_users(osv.osv):
      _name = 'res.users'
      _inherit = 'res.users'
  
 -    def copy(self, cr, uid, ids, default=None, context=None):
++    def copy_data(self, cr, uid, ids, default=None, context=None):
+         if default is None:
+             default = {}
 -        default = default.copy()
+         default.update({'employee_ids': False})
 -        return super(res_users, self).copy(cr, uid, ids, default, context=context)
++        return super(res_users, self).copy_data(cr, uid, ids, default, context=context)
+     
      def create(self, cr, uid, data, context=None):
          user_id = super(res_users, self).create(cr, uid, data, context=context)