Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-782168-uco
Revision history for this message
| Graeme Gellatly (gdgellatly) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
Hi,
I am so glad you went this way to fixing it, much more sensible, enforcing one pricelist per order.
However, there is a major bug in your code. What if the user changed the price manually? You will overwrite all his manual changes. I think you would be better implementing this in an onchange pricelist function.