Skip to Content
This question has been flagged
odoo12
2 Replies
1169 Views
Avatar
jenan soliman

hi,

I want to add a field in the hr. contract module called notification_id that will be calculated automatically according to the contract types.

here's my code:


class ContractNotificationInherit(models.Model):
    _inherit = 'hr.contract'

    notification_id = fields.Many2one('hr.contract.notification', string="renew notification for contract",
                                      required=True,
                                      compute="set_notification_group")

    @api.multi
    @api.depends('type_id')
    @api.model
    def set_notification_group(self):
        
        for rec in self.env['hr.contract'].search([]):
            if rec == self.env['hr.contract.type'].search([("name", '=', 'Employee')]):
                rec.notification_id = self.env['hr.contract.notification'].search([('name', '=', 'month')])

thank you in advance for any helps.




Avatar
Discard
Avatar
Waleed Ali Mohsen
Best Answer

You can use the below code for field compute method:

@api.depends('type_id')
def set_notification_group(self):
    for rec in self:
        if rec.type_id.name == "Employee":
            rec.notification_id = self.env['hr.contract.notification'].search([('name', '=', 'month')], limit=1).id

Avatar
Discard
Avatar
Cybrosys Techno Solutions Pvt.Ltd
Best Answer

Hi Jenan,

You can get the required notification record using the search method by applying the appropriate domain.Since you added compute method for the notification_id field you should iterate the self since there is a possibility of getting multiple objects in self.Then search the corresponding notification for each object in self and assign the value.Also you should assign a value for each record otherwise the compute method will raise an error.So initially you should assign the False value for every record.When you are searching the notification using search method by applying the particular domain there is a chance of getting multiple records, so you should limit the result by one.

Eg:

@api.depends('type_id')
def set_notification_group(self):
   for rec in self:
    rec.notification_id = False
       if rec.type_id.name == "Employee":
           notification = self.env['hr.contract.notification'].search([('name', '=', 'month')], limit=1).id
rec.notification_id = notification.id

Regards

Avatar
Discard
Related Posts Replies Views Activity
Odoo Mail Sending Limit Solved
odoo12
Avatar
Avatar
Avatar
2
Dec 23
12010
(Document type: Invoice, Operation: write) - (Records: [], User: 2)
odoo12
Avatar
0
Oct 23
33
Error while importing data in Odoo12: An unknown issue occurred during import (possibly lost connection, data limit exceeded or memory limits exceeded)
odoo12
Avatar
Avatar
Avatar
Avatar
3
Oct 23
787
Remove duplicate record when importing data from excel to Odoo
odoo12
Avatar
Avatar
1
Oct 23
569
Multiple group on field Odoo12
odoo12
Avatar
Avatar
1
Aug 23
982