Skip to Content
This question has been flagged
v8methodfields.functionodoo
3 Replies
4747 Rodiniai
Portretas
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

Portretas
Atmesti
Portretas
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.

Portretas
Atmesti
Portretas
Sairam Bandaru
Autorius 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

Portretas
Atmesti
Akhil P Sivan

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

Sairam Bandaru
Autorius

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
Autorius

Thanks Akhil.

Related Posts Replies Rodiniai Veikla
How to insert line break ?
v8 placeholder odoo
Portretas
Portretas
Portretas
Portretas
3
vas. 25
18470
How to use the "or" operator in functions(/methods)
python method odoo
Portretas
Portretas
Portretas
2
spal. 21
5061
[Error] : _prepare_invoice_line() takes exactly 5 arguments (3 given)
method arguments odoo
Portretas
0
lapkr. 16
4638
[8.0]sale_quotation_number: Error "undefined get method" when updating the amount
v8 error odoo
Portretas
Portretas
1
liep. 16
5279
Ajout d'une liste déroulante Odoo 8
v8 dropdown odoo
Portretas
0
liep. 16
6184