openerp-community-reviewer team mailing list archive

[Nhomar - Vauxoo <nhomar@xxxxxxxxx>]

Review: Needs Fixing

Hello, More comments.

In line #159

Why you didn't use python/openobject introspection to get the field definition, and they are wired in the code?, IMHO it is not a good approach.

Regards.
-- 
https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-account_financial_report_webkit_xls/+merge/192246
Your team Account Report Core Editors is subscribed to branch lp:account-financial-report.