on_change event does not pass context to the method!

[Dharmesh PATEL@Axelor SARL]

I wrote on_change event on product_id field in my object form to fill up another 'Description' field value with product name but what I notice it product name can not fetch according to language selected in preferences it fetch only English name.

    def onchange_product_id(self, cr, uid, ids, product_id, context={}):
        res = {'desc': ''}
        if product_id:
            product = self.pool.get('product.product').read(cr, uid, product_id, ['name'], context)
            res['desc'] = product['name'] and product['name'] or ''
        return {'value':res}

I also try to print context which I passed in 'onchange_product_id' and its print blank dictionary and there for it can not pass in product object's read method and product name cant fetch according to context language.

[Dharmesh PATEL@Axelor SARL (dpa-axelor)]

I am using latest stable version

[Numérigraphe (numerigraphe)]

If I'm not misunderstanding, I'm afraid you can't fix this bug without
fixing that one first:
https://bugs.launchpad.net/openobject-addons/+bug/535174
And I suspect there are plenty of other similar bugs.
Lionel

[Numérigraphe (numerigraphe)]

On a side note, I humbly suggest you use None as a default context as stated in https://bugs.launchpad.net/openobject-server/+bug/525808.
Lionel

[Stephane Wirtel (OpenERP) (stephane-openerp)]

The default value of the context argument in the declaration of the function must be None.

I confirm this point !

We will refuse the future code with context={} or any others variables.

[Stephane Wirtel (OpenERP) (stephane-openerp)]

Hi all,

I disagree with this feature request for the stable version.

Why.
1. Add the support of the context in the framework
2. Add the support of the context in the clients (Web and GTK, and KOO (maybe))
3. If we want to pass the context to the on_change function, we have to change the other "on_change" functions in the entire code of OpenERP addons/extra-addons

[Numérigraphe (numerigraphe)]

I marked other bugs as duplicates of this one since you agree to make the change in the whole code base.
Interesting comments had already been posted on those bugs, please make sure you read them.
Lionel.

[Stephane Wirtel (OpenERP) (stephane-openerp)]

Lionel,

I confirm for Trunk, not for stable.

[Olivier Dony (Odoo) (odo-openerp)]

I propose to drop this request for 6.0, as it's out of the scope and would imply a risky refactoring of all addons.
Instead, we plan to implement a better and more extensible mechanism for on_change parameters after v6.0, and it makes sense to do this at the same time. This is part of the product backlog post-v6, so it won't be forgotten.

[Hannes (Neobis) (nb2)]

in 7.0 you have to add context as a parameter in the xml view field.