openerp-community-reviewer team mailing list archive

["Luc De Meyer \(Noviat\)" <luc.demeyer@xxxxxxxxx>]

We have installed this module in the meanwhile at most of our customers and requirements vary from customer to customer which makes it difficult to decide upon the default fields but adding journal & analytic certainly makes sense. 
If no objection, I propose to change the default as follows : 

Current default : 

    def _report_xls_fields(self, cr, uid, context=None):
        return [
            'move', 'name', 'date', 'period', 'partner', 'account',
            'date_maturity', 'debit', 'credit', 'balance', 'reconcile', 'reconcile_partial',
        ]

New default:

    def _report_xls_fields(self, cr, uid, context=None):
        return [
            'move', 'name', 'date', 'journal', 'period', 'partner', 'account',
            'date_maturity', 'debit', 'credit', 'balance', 'reconcile', 'reconcile_partial', 'analytic_account',
        ]

I'll also update the module documentation so that anyone understands the customisation capabilities without having to read the code. 

Regards,
Luc

-----Original Message-----
From: bounces@xxxxxxxxxxxxx [mailto:bounces@xxxxxxxxxxxxx] On Behalf Of Alexis de Lattre
Sent: dinsdag 7 januari 2014 22:39
To: mp+192250@xxxxxxxxxxxxxxxxxx
Subject: Re: [Merge] lp:~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls into lp:account-financial-report

Yes, it is available, but it is not displayed by default in the XLS report, cf the fonction _report_xls_fields().

I think that we should add 2 fields in the default report :
- analytic account
- journal code

What do you think ?
-- 
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.