Code review comment for lp:~camptocamp/stock-logistic-flows/7.0-add_stock_picking_priority-afe

Revision history for this message
Sandy Carter (http://www.savoirfairelinux.com) (sandy-carter) wrote :

__openerp__: pep8 issues
 * spaces before the colon on l.56,57,58,59,66,67,68,71

picking_priority.py:
 * l.123 you're not checking if ids is an int or a list, this could lead to a problem with search and an error on l.137

review: Needs Fixing (code review)

« Back to merge proposal