Additional payment, bank statement and voucher addons for OpenERP

Code review comment for lp:~agilebg/account-payment/adding_account_due_list_7

Revision history for this message

Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote on 2013-12-20:

Hello Alex,

Here are some little things you should fixes before we can merge your contribution:

missing context propagation in browse and search method at:
l.121
l.123
l.138
l.148
l.228

Please add a Contributors section in __openerp__.py description

l.218 in fields_view_get context default value should be None instead of {}

Otherwise LGTM thanks for your contrib!

Cheers

review: Needs Fixing (code review, no tests)