Alexis, We already fixed this problems in the version that you find on our V7 "apps.openerp.com" launchpad branch, cf. ~noviat/openobject-addons/extra-7.0. This version also extends the inherit concept to add new fields (in order to support add-on modules that add account_move_line fields and also add french and dutch translations).
I have uploaded the new version to https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls/ Regards, Luc -----Original Message----- From: boun...@canonical.com [mailto:boun...@canonical.com] On Behalf Of Alexis de Lattre Sent: zaterdag 14 december 2013 0:17 To: mp+192...@code.launchpad.net Subject: Re: [Merge] lp:~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls into lp:account-financial-report About the crash that I mentionned 2 comments above, the reason is quite obvious when you look at the code in report/move_line_list_xls.py in the function generate_xls_report() : the variable debit_pos is declared and initialized in a block of code under : if 'balance' in wanted_list': and then it is used in a block of code which is executed in all cases. -- https://code.launchpad.net/~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls/+merge/192250 You are the owner of lp:~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls. 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. -- 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
