openerp-community-reviewer team mailing list archive

[Alexis de Lattre <alexis@xxxxxxxxxx>]

Review: Needs Fixing test

It is definitely a great module.

But I just found a bug : if you inherit _report_xls_fields() to personnalise the list of fields and you don't put the field 'balance' in the list, then you get a crash on report generation :

Traceback (most recent call last):
  File "/home/erp_super/erp/prod/src/server/openerp/service/web_services.py", line 712, in go
    (result, format) = obj.create(cr, uid, ids, datas, context)
  File "/home/erp_super/erp/prod/src/reporting-engines/report_xls/report_xls.py", line 106, in create
    return self.create_source_xls(cr, uid, ids, data, context)
  File "/home/erp_super/erp/prod/src/reporting-engines/report_xls/report_xls.py", line 122, in create_source_xls
    self.generate_xls_report(_p, _xs, data, objs, wb)
  File "/home/erp_super/erp/prod/src/custom_scentys/test-modules/account_move_line_report_xls/report/move_line_list_xls.py", line 181, in generate_xls_report
    debit_cell = rowcol_to_cell(row_pos, debit_pos)
UnboundLocalError: local variable 'debit_pos' referenced before assignment
-- 
https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls/+merge/192250
Your team Account Report Core Editors is subscribed to branch lp:account-financial-report.