[@api.onchange] change state fields value

Question

Hi Community
I wrote an onchange method that change the state field value based on the value of another field.

@ api.multi
@ api.onchange ( 'lgs' )
def on_change_state( self ):
values = {}
for record in  self :
if record.lgs:
values = { 'state' : 'waiting' }
else :
values = { 'state' : 'draft' }
print  self .state, self .lgs
return { 'value' : values}
I see that the "state" value change. But when I save the form the state field take the old version.
Please How can I resolve this issue. I think

Yenthe Van Ginneken (Mainframe Monkey) · Accepted Answer

Hi Othmane,

Why do you have @api.multi above the @api.onchange? They should not be combined, you only need the onchange.
As for the value assigning: why don't you simply write on self right away? Your code should look like this:
@api.onchange(lgs)
def on_change_state(self):
for record in self:
if record.lgs:
record.state = 'waiting'
else:
record.state = 'draft'
print self.state, self.lgs

Regards,
Yenthe

zahid · Answer

onchange method does not need to retrun a value.
you can do this
@api.multi
@api.onchange('lgs')
def on_change_state(self):
for record in self:
if record.lgs:
record.state = 'waiting'
else:
record.state = 'draft'

Mohammed Amal N · Answer

You are not writing values to self,
Try this * self.write(values)*

Othmane Ghandi · Answer

Mohammed Amal
I tried it but it didn't work.
I got the answer from a  colleague "Maniganda" : in my case I removed the " readonly=True " attribute of the state fields and it work !!

Related Posts	Odgovori	Prikazi
How to make a field readonly from a function? or Can we call a function on readonly attribute like "readonly="function_name" in odoo9? fields readonly onchange	1 sep. 16	10122
api.onchange not working on many2many - odoo 13 many2many onchange api.onchange Odoo13.0	1 avg. 24	3717
in contact module contacts state onchange salesperson	1 nov. 22	2874
Can't make a Selection field grab values from a function? Solved fields selection onchange compute	2 maj 22	15469
Error in calculated field fields properties api.depends api.onchange	1 apr. 21	4056

This question has been flagged

Enjoying the discussion? Don't just read, join in!