Skip to Content
This question has been flagged
fieldsstateonchangeapi.onchange
4 Replies
53933 Views
Avatar
Othmane Ghandi

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 this issue is only for state field
Avatar
Discard
Avatar
Yenthe Van Ginneken (Mainframe Monkey)
Best 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

Avatar
Discard
Othmane Ghandi
Author

@api.multi above the @api.onchange They should not be combined: You are right !! thanks !!

But it's working hhh

Avatar
zahid
Best 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'
Avatar
Discard
Avatar
Mohammed Amal N
Best Answer

You are not writing values to self,

Try this  self.write(values)

Avatar
Discard
Othmane Ghandi
Author

I tried it but it doesn't solve my problem :(

Avatar
Othmane Ghandi
Author Best 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 !!
 

Avatar
Discard
Related Posts Replies Views Activity
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
Avatar
Avatar
1
Sep 16
8535
api.onchange not working on many2many - odoo 13
many2many onchange api.onchange Odoo13.0
Avatar
Avatar
1
Aug 24
2252
in contact module
contacts state onchange salesperson
Avatar
Avatar
1
Nov 22
1855
Can't make a Selection field grab values from a function? Solved
fields selection onchange compute
Avatar
Avatar
Avatar
2
May 22
13883
Error in calculated field
fields properties api.depends api.onchange
Avatar
Avatar
1
Apr 21
2767