[6.0][sale] warning out of stock product screws the product_change event

[Raphaël Valyi - http://www.akretion.com]

Hello,

on latest trunk version.
Take fresh database with demo data and sale module installed.

Associate a sale tax to the "PC1" product.
Then create a new sale order where you will sale 1000 PC1.
You'll get a warning like "warning PC1 is out of stock"

This is cool to warn the user,
but the way this is implemented in v6 (with raise it seems) currently shortcuts all the product_id_change event and screw the assignation of the other fields such as UOM, taxes etc... In that case your PC1 line won't receive any sale tax which is a severe bug (indeed, despite we don't have the product in stock we might still register a proper order for it of course).

Please look how I implemented that in v5 with a warning message instead that do not screw the normal product_id_change variable assignation:
http://openerp.com/forum/topic17085.html?sid=e5a2b39933c953db4269a99229a2e40b

Hope this helps

[Jay Vora (Serpent Consulting Services) (jayvora)]

Hello Raphaël,

I configured Product PC1 with sale tax 'Tax 1'.

I tried to generate the error, but did not receive it.

Would you please try again?

Thanks.

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

Hello Jay,

I've not seen the bug today, may be it has been fixed by a very recent commit, will double check.

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

All right, seems to be fixed indeed, thanks!