Code review comment for lp:~acsone-openerp/account-financial-report/7.0-bug-1312732-lmi
Revision history for this message
| Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote | # |
review:
Needs Information
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
You are right forgot that sudden move from Odoo
At least it fixes this issue in 7.0 but this would be more effort when porting to 8.0.
Thus I revert my review as we need to find an other way of ensuring values are defined.
BTW shouldn't store be computed when adding the column on install?