literal string parameters to on_change not supported

may be that this happens only if the field with the trigger is new in the module

please increase the importance

We must be able to rely that triggers work

I believe the issue comes from the web client currently not handling literal strings for onchange parameters, more than anything related to inheritance

A fix for this issue has been pushed to the branch https://code.launchpad.net/~openerp-dev/openerp-web/trunk-onchange-string-literals-xmo

Could you test if it solves the problem on your custom module?

Fix pushed to trunk, revision <email address hidden>

Sorry the issue is definitely not fixed

the trigger writes a log text for test purpose, and it works fine with GTK, but not log text when using the web client rev 1528

hence the trigger is NOT fired at all.

> the trigger writes a log text for test purpose, and it works fine with GTK, but not log text when using the web client rev 1528

And yet adding the same onchange call your perform (a string and 2 variable arguments) to an existing view works perfectly here.

Could you ensure your cache has been cleaned (to know the code is up to date) and see (and copy here) any error appearing in your javascript console?

Failed to load resource
js:29070Error: Could not get field with name '{'company_id': parent.company_id}' for onchange 'product_id_change_c2c_pu(product_id, uos_id, quantity, name, parent.type, parent.partner_id, parent.fiscal_position, price_unit_pu, parent.address_invoice_id, parent.currency_id, {'company_id': parent.company_id}, price_unit_id)'

sorry
it's a coding error
my be the js errors could be shown in pop-ups

> sorry it's a coding error

No problem closing the bug as resolved then.

> my be the js errors could be shown in pop-ups

Yeah, making it easier to see would probably be a good idea, at least for this kind of errors.

Sorry once more

I changed (error) (1)
on_change="product_id_change_c2c_pu(product_id, uos_id, quantity, name, parent.type, parent.partner_id, parent.fiscal_position, price_unit_pu, parent.address_invoice_id, parent.currency_id, {'company_id': parent.company_id},price_unit_id)"

to (working) (2)
on_change="product_id_change_c2c_pu(product_id, uos_id, quantity, name, parent.type, parent.partner_id, parent.fiscal_position, price_unit_pu, parent.address_invoice_id, parent.currency_id, parent.company_id,price_unit_id)"/>

replacing
{'company_id': parent.company_id}
by
parent.company_id

nevertheless
parent.company_id is a non existing field
- hence the the coding (1) returned an error - which is correct behaviour
- while coding(2) just ignores a reference to a not existing field - which IMHO should not be allowed.

> parent.company_id is a non existing field

Yes, that does sound like a bug in the handling of parent.* references in on_change resolution. But it's a different issue than this bug, please stop reopening this one and create a new one for that new issue.

After a Google search, I arrived on this bug report. I am having some issues with the web interface of OpenERP 6.1 when passing parent.* fields to on_change. The bug occurs with Firefox but not with Chrome. I'll try to make a new bug report if I have time.