Code review comment for lp:~openerp-dev/openobject-addons/6.0-bug-857067-rha
Revision history for this message
| Raphael Collet (OpenERP) (rco-openerp) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
This is the wrong approach. Don't override do_partial().
The values of weight and weight_net are probably wrong because their stored value is not recomputed after some data change. Please fix the 'store' argument in the function fields! What is the change in the data that requires those fields to be recomputed?