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: boun...@canonical.com [mailto:boun...@canonical.com] On Behalf Of Alexis 
de Lattre
Sent: dinsdag 7 januari 2014 22:39
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

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.

-- 
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

Reply via email to