Review: Needs Fixing code review, no test
l.288 looks like you forgot to put anything in for group_id:id
purchase_profit_forecast/__openerp__.py:
Manifest "license" key is missing
Deprecated Manifest key ('active')
Deprecated Manifest key ('certificate')
Flake8:
report/analytic.py:3:1: E302 expected 2 blank lines, found 1
report/analytic.py:10:80: E501 line too long (80 > 79 characters)
report/purchase.py:4:1: E302 expected 2 blank lines, found 1
report/purchase.py:38:5: E303 too many blank lines (2)
report/purchase.py:41:9: F841 local variable 'result_context' is assigned to
but never used
report/purchase.py:54:1: W391 blank line at end of file
Copyright dates are wrong
No .pot file
No unittests
--
https://code.launchpad.net/~savoirfairelinux-openerp/purchase-wkfl/purchase_profit_forecast-missing-security-access/+merge/218509
Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl.
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help : https://help.launchpad.net/ListHelp
