Skip to Content
Menu
This question has been flagged
3 Replies
4381 Views

Hello,

Im trying to create the method which has to check the record has selected or not in one2many tree view. If the record has selected then the method will change state. If not, then it will raise the warning for user.

model_b.py :

name = fields.Many2one('hr.employee')

o_2_m = fields.One2many('model.a', 'keya')

state = fields.Selection([

            ('draf', 'DRAF'),

            ('confirm', 'CONFIRMED'),

            (cancel', 'CANCEL'),

            ],default='draf')


model_a.py :

name = fields.Many2one('hr.employee')

keya = fields.Many2one('model.b')


The method's code like this:

def confirm(self)

    for rec in self:

            mod_b = self.env["model.b"].search([('o_2_m.name', '=', rec.name.id)])

            mod_a = self.env["model.a"].search([('name', '=', rec.name.id)])

            if mod_b.o_2_m.name == mod_a.name :

                mod_b.write({'state': 'confirm'})                

            else:

                return {

                    'warning': {

                        'title': "You forget to select the record for changing in o2m",

                        'message': "Please select as below",

                    },

                }

I have tested with select no select the record via "Add a line" link in one2many tree view of model_b.py. But nothing works.


Please help!

Thank you!

Avatar
Discard
Best Answer

Hi, 

I hope the change of state function must be in model_b.py and add this as a button function in your xml view of model_b. 

Here you can raise error or warning and also move state as you required. Just give a try

from odoo.exceptions import UserError

@api.multi

def confirm(self)

    for rec in self:

            if not rec.o_2_m:

                 raise UserError(_("You forget to select the record for changing in o2m"\nPlease select as below"))               

            rec.state = 'confirm'               

     return True

Avatar
Discard

I changed the function to the model_b because searching a model takes much time when compared to accessing the fields with "." (Dot) . Just for code optimization.

Author

Yes, the change of state in model_b.py. And your code is really good. Thank you, Karthikeyan

Best Answer

Hi,

Try the below code and see whether it helps,

from odoo.exceptions import ValidationError

def confirm(self):
for rec in self:
mod_b = self.env["model.b"].search([('o_2_m.name', '=', rec.name.id)])
if rec.o_2_m:
for line in rec.o_2_m:
line.write({'state': 'confirm'})
else:
raise ValidationError(_('You forget to select the record for changing in o2m.'))


Reference:

Odoo ORM Methods

Thanks

Avatar
Discard
Author

Hi Niyas. Thank you for your support. But the state selection is from the model_b.py. But in your code, the state change via the o2m of model a. And I got the error inform from this.

Author Best Answer

I'd resolved my wish, base on Niyas's support.

The code looks like this:

from odoo.exceptions import ValidationError

def confirm(self):
for rec in self:
mod_b = self.env["model.b"].search([('o_2_m.name', '=', rec.name.id)])
if rec.o_2_m:
          mod_b_state = self.env["model.b"].search([])
           if mod_b_state:
              mod_b_state.write({'state': 'confirm'})
else:
raise ValidationError(_('You forget to select the record for changing in o2m.'))
Avatar
Discard
Related Posts Replies Views Activity
0
Feb 21
2913
2
Oct 20
1777
1
Aug 19
9187
1
Jul 22
2794
1
Dec 20
277