Skip to Content
This question has been flagged
v8methodfields.functionodoo
3 Replies
3643 Views
Avatar
Sairam Bandaru

Hi

I want inherit these field to modify the result in functional field but inherited function not calling


class hr_employee_inherit(osv.osv):
 _inherit = 'hr.employee'

 def _get_leave_status(self, cr, uid, ids, name, args, context=None):
 holidays_obj = self.pool.get('hr.holidays')
 holidays_id = holidays_obj.search(cr, uid,
 [('employee_id', 'in', ids), ('leave_from','<=',fields.date.context_today),
 ('leave_to','>=',time.strftime('%Y-%m-%d')),('type','=','remove'),('state','not in',('cancel','refuse'))],
 context=context)
 print holidays_id,ids,'holidays_id****In INHERIT***********-------------------------',time.strftime('%Y-%m-%d'),fields.date.context_today
 result = {}
 for id in ids:
 result[id] = {
 'current_leave_state': False,
 'current_leave_id': False,
 'leave_date_from':False,
 'leave_date_to':False,
 }
 for holiday in self.pool.get('hr.holidays').browse(cr, uid, holidays_id, context=context):
 result[holiday.employee_id.id]['leave_date_from'] = holiday.date_from
 result[holiday.employee_id.id]['leave_date_to'] = holiday.date_to
 result[holiday.employee_id.id]['current_leave_state'] = holiday.state
 result[holiday.employee_id.id]['current_leave_id'] = holiday.holiday_status_id.id
 print result,'Reslu leave status---------------******-----------' 
 return result

 _columns = {
 'current_leave_id': fields.function(_get_leave_status, multi="leave_status", string="Current Leave Type",type='many2one', relation='hr.holidays.status'),
 'leave_date_from': fields.function(_get_leave_status, multi='leave_status', type='date', string='From Date'),
 'leave_date_to': fields.function(_get_leave_status, multi='leave_status', type='date', string='To Date'),
 }


help me out


thanks

Avatar
Discard
Avatar
Akhil P Sivan
Best Answer

Hi,

If you want to inherit the functional fields and make some changes, you may redefine the fields and function in new api and make the changes you need. For eg, you may try like this:

class hr_employee(models.Model):
    _inherit="hr.employee"
    
    current_leave_id = fields.Many2one('hr.holidays.status', compute='_get_leave_status', string="Current Leave Type")
    leave_date_from = fields.Date(compute='_get_leave_status', string='From Date')
    leave_date_to = fields.Date(compute='_get_leave_status', string='To Date')
    
    
    @api.one
    @api.depends('employee_id')
    def _get_leave_status(self):
        holidays_obj = self.env['hr.holidays']
        holidays = holidays_obj.search([('employee_id', 'in', self.ids), ('date_from','<=',time.strftime('%Y-%m-%d %H:%M:%S')),
           ('date_to','>=',time.strftime('%Y-%m-%d 23:59:59')),('type','=','remove'),('state','not in',('cancel','refuse'))])

        for holiday in holidays:
            self.leave_date_from = holiday.date_from
            self.leave_date_to = holiday.date_to
            self.current_leave_state = holiday.state
            self.current_leave_id = holiday.holiday_status_id.id

Please check it n let me know.

Avatar
Discard
Avatar
Sairam Bandaru
Author Best Answer

Hi, that's fine now able to inherit but in the function I have changed main code because 'self' contained all employees ,So I looped on self instead of holidays as per this code it raising error, So I modified like this working fine  

@api.multi

def _get_leave_status(self):

holidays_obj = self.env['hr.holidays']

print self.ids,'holidays_id****In INHERIT***********-------------------------',fields.date.context_today(self,self.env.cr,self.env.uid)

print self.env.context,'Context ***********8-------------------------',self,self.env.cr,self.env.uid

holidays_id = holidays_obj.search(

[('employee_id', 'in', self.ids), ('leave_from','<=',fields.date.context_today(self,self.env.cr,self.env.uid)),

('leave_to','>=',fields.date.context_today(self,self.env.cr,self.env.uid)),('type','=','remove'),('state','not in',('cancel','refuse'))],

)

print holidays_id,'holidays_id****In INHERIT***********-------------------------',fields.date.context_today(self,self.env.cr,self.env.uid)

#result = {}

'''

for id in ids:

result[id] = {

'current_leave_state': False,

'current_leave_id': False,

'leave_date_from':False,

'leave_date_to':False,

}

'''

for holiday in holidays_id:

print holiday,'****************************',holiday.holiday_status_id.id

for rec in self:

print rec.ids,'Rec*88888888888888888888'

holidays_id = holidays_obj.search(

[('employee_id', 'in', rec.ids), ('leave_from','<=',fields.date.context_today(self,self.env.cr,self.env.uid)),

('leave_to','>=',fields.date.context_today(self,self.env.cr,self.env.uid)),('type','=','remove'),('state','not in',('cancel','refuse'))],

)

rec.leave_date_from = holidays_id and holidays_id[0].date_from or None

rec.leave_date_to = holidays_id and holidays_id[0].date_to or None

rec.current_leave_state = holidays_id and holidays_id[0].state or None

rec.current_leave_id = holidays_id and holidays_id[0].holiday_status_id.id or None


Thanks

Avatar
Discard
Akhil P Sivan

Hi did you manage to solve it? Is it working now?

Sairam Bandaru
Author

Yes working fine.Resolved I inherited fields along with function as computed fields Thanks.

Akhil P Sivan

Instead of looping on self, I used "@api.one" decorator. Both should have same effect. Anyway I didn't get time to check by running the above. Hope, it helped you in solving the problem.

Sairam Bandaru
Author

Thanks Akhil.

Related Posts Replies Views Activity
How to insert line break ?
v8 placeholder odoo
Avatar
Avatar
Avatar
Avatar
3
Mar 24
14738
How to use the "or" operator in functions(/methods)
python method odoo
Avatar
Avatar
Avatar
2
Oct 21
2215
[Error] : _prepare_invoice_line() takes exactly 5 arguments (3 given)
method arguments odoo
Avatar
0
Nov 16
3220
[8.0]sale_quotation_number: Error "undefined get method" when updating the amount
v8 error odoo
Avatar
Avatar
1
Jul 16
2858
Ajout d'une liste déroulante Odoo 8
v8 dropdown odoo
Avatar
0
Jul 16
4131