Hi, I propose to clean up the on_change functions signature. Currently, it is like this:
def on_change_xxx(self, cursor, user, ids, vals, context=None):
pass
But as on_change could happen on non saved records, the ids list is not
usable and must never be used because the data from the database doesn't
match the data in the client. on_change function must only rely on values of
vals. I made a rapid check of the current module and it seems it is already
the case. So I propose to change the signature into this:
def on_change_xxx(self, cursor, user, vals, context=None):
pass
--
Cédric Krier
B2CK SPRL
Rue de Rotterdam, 4
4000 Liège
Belgium
Tel: +32 472 54 46 59
Email/Jabber: [email protected]
Website: http://www.b2ck.com/
pgpQBeD81v7tz.pgp
Description: PGP signature
