Code review comment for lp:~akretion-team/account-invoicing/70-add-invoice_fiscal_position_update

Revision history for this message
Alexis de Lattre (alexis-via) wrote :

Dear OpenERP community friends,

Here it is, I manage to make the "on_change" solution work, thanks to the function "resolve_2many_commands" pointed out by Graeme. I tested it and it seems to work well in all scenarios (at least all the scenarios that I tested ! :). Please test and report issues if you find some.

It's quite a big change in the code and it's the first time that I use the function "resolve_2many_commands", so tell me what you think of the new code.

Enjoy,

Alexis

« Back to merge proposal