From 46f017c53b9303a4e6d254c1b3d2560e5904c3e9 Mon Sep 17 00:00:00 2001 From: Humberto Arocha Date: Thu, 25 Jul 2013 18:26:24 +0000 Subject: [PATCH 1/5] [IMP] Field target_move is added to wizard in afr so that it will used in the future to filter posted and unposted entries --- account_financial_report/view/wizard.xml | 1 + account_financial_report/wizard/wizard.py | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/account_financial_report/view/wizard.xml b/account_financial_report/view/wizard.xml index 4136bfb1..76deeaff 100644 --- a/account_financial_report/view/wizard.xml +++ b/account_financial_report/view/wizard.xml @@ -47,6 +47,7 @@ + diff --git a/account_financial_report/wizard/wizard.py b/account_financial_report/wizard/wizard.py index be9ff1eb..8e392f09 100644 --- a/account_financial_report/wizard/wizard.py +++ b/account_financial_report/wizard/wizard.py @@ -54,6 +54,9 @@ class wizard_report(osv.osv_memory): 'tot_check': fields.boolean('Summarize?', help='Checking will add a new line at the end of the Report which will Summarize Columns in Report'), 'lab_str': fields.char('Description', help='Description for the Summary', size=128), + 'target_move': fields.selection([('posted', 'All Posted Entries'), + ('all', 'All Entries'), + ], 'Target Moves', help='Print All Accounting Entries or just Posted Accounting Entries'), #~ Deprecated fields 'filter': fields.selection([('bydate', 'By Date'), ('byperiod', 'By Period'), ('all', 'By Date and Period'), ('none', 'No Filter')], 'Date/Period Filter'), 'date_to': fields.date('End date'), @@ -70,6 +73,7 @@ class wizard_report(osv.osv_memory): 'fiscalyear': lambda self, cr, uid, c: self.pool.get('account.fiscalyear').find(cr, uid), 'display_account': lambda *a: 'bal_mov', 'columns': lambda *a: 'five', + 'target_move': 'posted', } def onchange_inf_type(self, cr, uid, ids, inf_type, context=None): From 8d57b7a79b9d9bd1bf98db5a21d8de995c9a5c70 Mon Sep 17 00:00:00 2001 From: Humberto Arocha Date: Thu, 25 Jul 2013 18:34:18 +0000 Subject: [PATCH 2/5] [IMP] Field target_move is added to template in afr so that it will used in the future to filter posted and unposted entries --- account_financial_report/model/account_financial_report.py | 4 ++++ .../view/account_financial_report_view.xml | 1 + 2 files changed, 5 insertions(+) diff --git a/account_financial_report/model/account_financial_report.py b/account_financial_report/model/account_financial_report.py index 46acb7d0..12d23aaa 100644 --- a/account_financial_report/model/account_financial_report.py +++ b/account_financial_report/model/account_financial_report.py @@ -51,6 +51,9 @@ class account_financial_report(osv.osv): 'tot_check': fields.boolean('Summarize?', help='Checking will add a new line at the end of the Report which will Summarize Columns in Report'), 'lab_str': fields.char('Description', help='Description for the Summary', size=128), + 'target_move': fields.selection([('posted', 'All Posted Entries'), + ('all', 'All Entries'), + ], 'Target Moves', help='Print All Accounting Entries or just Posted Accounting Entries'), #~ Deprecated fields 'filter': fields.selection([('bydate', 'By Date'), ('byperiod', 'By Period'), ('all', 'By Date and Period'), ('none', 'No Filter')], 'Date/Period Filter'), @@ -69,6 +72,7 @@ class account_financial_report(osv.osv): 'date_from': lambda *a: time.strftime('%Y-%m-%d'), 'date_to': lambda *a: time.strftime('%Y-%m-%d'), 'filter': lambda *a: 'byperiod', + 'target_move': 'posted', } def copy(self, cr, uid, id, defaults, context=None): diff --git a/account_financial_report/view/account_financial_report_view.xml b/account_financial_report/view/account_financial_report_view.xml index 40cd1d8f..6306ba76 100644 --- a/account_financial_report/view/account_financial_report_view.xml +++ b/account_financial_report/view/account_financial_report_view.xml @@ -30,6 +30,7 @@ + Date: Thu, 25 Jul 2013 20:29:21 +0000 Subject: [PATCH 3/5] [IMP] Now reports different From General Ledger can show posted or all entries depending on the selection. --- account_financial_report/report/parser.py | 1 + 1 file changed, 1 insertion(+) diff --git a/account_financial_report/report/parser.py b/account_financial_report/report/parser.py index ebb74b92..16329078 100644 --- a/account_financial_report/report/parser.py +++ b/account_financial_report/report/parser.py @@ -321,6 +321,7 @@ class account_balance(report_sxw.rml_parse): def z(n): return abs(n) < 0.005 and 0.0 or n + self.context['state'] = form['target_move'] self.from_currency_id = self.get_company_currency(form['company_id'] and type(form[ 'company_id']) in (list, tuple) and form['company_id'][0] or form['company_id']) if not form['currency_id']: From b03c6b79302fc5e2f28e074a70cdccae5102df9e Mon Sep 17 00:00:00 2001 From: Humberto Arocha Date: Thu, 25 Jul 2013 23:06:44 +0000 Subject: [PATCH 4/5] [FIX] Totalizing lines is being fixed and now summarize only those lines which are in the account_list. --- account_financial_report/report/parser.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/account_financial_report/report/parser.py b/account_financial_report/report/parser.py index 16329078..e4d4c0c1 100644 --- a/account_financial_report/report/parser.py +++ b/account_financial_report/report/parser.py @@ -332,6 +332,7 @@ class account_balance(report_sxw.rml_parse): if 'account_list' in form and form['account_list']: account_ids = form['account_list'] + account_list= form['account_list'] del form['account_list'] credit_account_ids = self.get_company_accounts(form['company_id'] and type(form[ @@ -726,11 +727,10 @@ class account_balance(report_sxw.rml_parse): # # Check whether we must sumarize this line in the report or not # - if form['tot_check'] and res['type'] == 'view' and res['level'] == 1 and (res['id'] not in tot): - + if form['tot_check'] and (res['id'] in account_list) and (res['id'] not in tot): if form['columns'] == 'qtr': tot_check = True - #~ tot[res['id']] = True + tot[res['id']] = True tot_bal1 += res.get('bal1', 0.0) tot_bal2 += res.get('bal2', 0.0) tot_bal3 += res.get('bal3', 0.0) @@ -739,7 +739,7 @@ class account_balance(report_sxw.rml_parse): elif form['columns'] == 'thirteen': tot_check = True - #~ tot[res['id']] = True + tot[res['id']] = True tot_bal1 += res.get('bal1', 0.0) tot_bal2 += res.get('bal2', 0.0) tot_bal3 += res.get('bal3', 0.0) @@ -755,7 +755,7 @@ class account_balance(report_sxw.rml_parse): tot_bal13 += res.get('bal13', 0.0) else: tot_check = True - #~ tot[res['id']] = True + tot[res['id']] = True tot_bin += res['balanceinit'] tot_deb += res['debit'] tot_crd += res['credit'] From 14af492ba113bed3f2decbb833918abdbedac454 Mon Sep 17 00:00:00 2001 From: Humberto Arocha Date: Thu, 25 Jul 2013 23:47:03 +0000 Subject: [PATCH 5/5] [IMP] Now Analytic lines is aware of the option 'Entries to Include' --- account_financial_report/model/account_financial_report.py | 3 ++- account_financial_report/report/parser.py | 6 ++++-- account_financial_report/wizard/wizard.py | 3 ++- 3 files changed, 8 insertions(+), 4 deletions(-) diff --git a/account_financial_report/model/account_financial_report.py b/account_financial_report/model/account_financial_report.py index 12d23aaa..b1e50d68 100644 --- a/account_financial_report/model/account_financial_report.py +++ b/account_financial_report/model/account_financial_report.py @@ -53,7 +53,8 @@ class account_financial_report(osv.osv): 'lab_str': fields.char('Description', help='Description for the Summary', size=128), 'target_move': fields.selection([('posted', 'All Posted Entries'), ('all', 'All Entries'), - ], 'Target Moves', help='Print All Accounting Entries or just Posted Accounting Entries'), + ], 'Entries to Include', required=True, + help='Print All Accounting Entries or just Posted Accounting Entries'), #~ Deprecated fields 'filter': fields.selection([('bydate', 'By Date'), ('byperiod', 'By Period'), ('all', 'By Date and Period'), ('none', 'No Filter')], 'Date/Period Filter'), diff --git a/account_financial_report/report/parser.py b/account_financial_report/report/parser.py index e4d4c0c1..cd534ea0 100644 --- a/account_financial_report/report/parser.py +++ b/account_financial_report/report/parser.py @@ -181,7 +181,8 @@ class account_balance(report_sxw.rml_parse): #~ periods = str(tuple(ctx['periods'])) where = """where aml.period_id in (%s) and aa.id = %s and aml.state <> 'draft'""" % ( periods, account['id']) - + if ctx.get('state','posted')=='posted': + where += "AND am.state = 'posted'" sql_detalle = """select aml.id as id, aj.name as diario, aa.name as descripcion, (select name from res_partner where aml.partner_id = id) as partner, aa.code as cuenta, aml.name as name, @@ -321,7 +322,8 @@ class account_balance(report_sxw.rml_parse): def z(n): return abs(n) < 0.005 and 0.0 or n - self.context['state'] = form['target_move'] + self.context['state'] = form['target_move'] or 'posted' + self.from_currency_id = self.get_company_currency(form['company_id'] and type(form[ 'company_id']) in (list, tuple) and form['company_id'][0] or form['company_id']) if not form['currency_id']: diff --git a/account_financial_report/wizard/wizard.py b/account_financial_report/wizard/wizard.py index 8e392f09..b15d54f2 100644 --- a/account_financial_report/wizard/wizard.py +++ b/account_financial_report/wizard/wizard.py @@ -56,7 +56,8 @@ class wizard_report(osv.osv_memory): 'target_move': fields.selection([('posted', 'All Posted Entries'), ('all', 'All Entries'), - ], 'Target Moves', help='Print All Accounting Entries or just Posted Accounting Entries'), + ], 'Entries to Include', required=True, + help='Print All Accounting Entries or just Posted Accounting Entries'), #~ Deprecated fields 'filter': fields.selection([('bydate', 'By Date'), ('byperiod', 'By Period'), ('all', 'By Date and Period'), ('none', 'No Filter')], 'Date/Period Filter'), 'date_to': fields.date('End date'),