Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-871314-bde
Revision history for this message
| Mustufa Rangwala (Open ERP) (mra-tinyerp) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
Bharat,
Try to follow python coding standard, declare pool variable at beginning of method and pass context in write method too.
Above changes will make your Merge proposal more consistent.
Thanks,
Mustufa