diff --git a/account_financial_report_webkit/report/common_reports.py b/account_financial_report_webkit/report/common_reports.py index 11c4203d..fbbf1e5a 100644 --- a/account_financial_report_webkit/report/common_reports.py +++ b/account_financial_report_webkit/report/common_reports.py @@ -492,23 +492,14 @@ class CommonReportHeaderWebkit(common_report_header): raise osv.except_osv( _('No valid filter'), _('Please set a valid time filter')) - def _get_move_line_datas(self, move_line_ids, - order='per.special DESC, l.date ASC, \ - per.date_start ASC, m.name ASC'): - # Possible bang if move_line_ids is too long - # We can not slice here as we have to do the sort. - # If slice has to be done it means that we have to reorder in python - # after all is finished. That quite crapy... - # We have a defective desing here (mea culpa) that should be fixed - # - # TODO improve that by making a better domain or if not possible - # by using python sort - if not move_line_ids: - return [] - if not isinstance(move_line_ids, list): - move_line_ids = [move_line_ids] - monster = """ -SELECT l.id AS id, + def _get_move_line_select(self): + ''' + Get the columns to put in the SQL SELECT for _get_move_line_datas + + See _get_move_line_datas for available tables and aliases + ''' + return """ + l.id AS id, l.date AS ldate, j.code AS jcode , j.type AS jtype, @@ -534,18 +525,46 @@ SELECT l.id AS id, i.type AS invoice_type, i.number AS invoice_number, l.date_maturity -FROM account_move_line l - JOIN account_move m on (l.move_id=m.id) - LEFT JOIN res_currency c on (l.currency_id=c.id) - LEFT JOIN account_move_reconcile partialrec - on (l.reconcile_partial_id = partialrec.id) - LEFT JOIN account_move_reconcile fullrec on (l.reconcile_id = fullrec.id) - LEFT JOIN res_partner p on (l.partner_id=p.id) - LEFT JOIN account_invoice i on (m.id =i.move_id) - LEFT JOIN account_period per on (per.id=l.period_id) - JOIN account_journal j on (l.journal_id=j.id) - WHERE l.id in %s""" - monster += (" ORDER BY %s" % (order,)) + """ + + def _get_move_line_order(self): + ''' Get the default SQL ORDER statement for _get_move_line_datas ''' + return 'per.special DESC, l.date ASC, per.date_start ASC, m.name ASC' + + def _get_move_line_datas(self, move_line_ids, order=None): + if order is None: + order = self._get_move_line_order() + # Possible bang if move_line_ids is too long + # We can not slice here as we have to do the sort. + # If slice has to be done it means that we have to reorder in python + # after all is finished. That quite crapy... + # We have a defective desing here (mea culpa) that should be fixed + # + # TODO improve that by making a better domain or if not possible + # by using python sort + if not move_line_ids: + return [] + if not isinstance(move_line_ids, list): + move_line_ids = [move_line_ids] + monster = """ + SELECT {select} + FROM account_move_line l + JOIN account_move m on (l.move_id=m.id) + LEFT JOIN res_currency c on (l.currency_id=c.id) + LEFT JOIN account_move_reconcile partialrec + ON (l.reconcile_partial_id = partialrec.id) + LEFT JOIN account_move_reconcile fullrec + ON (l.reconcile_id = fullrec.id) + LEFT JOIN res_partner p on (l.partner_id=p.id) + LEFT JOIN account_invoice i on (m.id =i.move_id) + LEFT JOIN account_period per on (per.id=l.period_id) + JOIN account_journal j on (l.journal_id=j.id) + WHERE l.id in %s + ORDER BY {order} + """.format( + select=self._get_move_line_select(), + order=order, + ) try: self.cursor.execute(monster, (tuple(move_line_ids),)) res = self.cursor.dictfetchall() diff --git a/account_financial_report_webkit_xls/report/general_ledger_xls.py b/account_financial_report_webkit_xls/report/general_ledger_xls.py index 666cee75..3b3b35ab 100644 --- a/account_financial_report_webkit_xls/report/general_ledger_xls.py +++ b/account_financial_report_webkit_xls/report/general_ledger_xls.py @@ -30,10 +30,22 @@ from openerp.tools.translate import _ # import logging # _logger = logging.getLogger(__name__) + +class GeneralLedgerWebkitSupplier(GeneralLedgerWebkit): + """ Extends General Ledger Parser to add the supplier invoice + number in the move lines """ + + def _get_move_line_select(self): + res = super(GeneralLedgerWebkitSupplier, self)._get_move_line_select() + return res + """ + , i.supplier_invoice_number AS supplier_invoice_number + """ + _column_sizes = [ ('date', 12), ('period', 12), ('move', 20), + ('supp_inv_no', 22), ('journal', 12), ('account_code', 12), ('partner', 30), @@ -46,6 +58,17 @@ _column_sizes = [ ('curr_code', 7), ] +# Header column spans +COLS_COA = 3 +COLS_FY = 1 +COLS_DF = 3 +COLS_AF = 1 +COLS_TM = 2 +COLS_IB = 2 + +# Full column span +COLS_TOT = sum((COLS_COA, COLS_FY, COLS_DF, COLS_AF, COLS_TM, COLS_IB)) + class general_ledger_xls(report_xls): column_sizes = [x[1] for x in _column_sizes] @@ -74,7 +97,7 @@ class general_ledger_xls(report_xls): _p.company.partner_id.name, _p.company.currency_id.name]) c_specs = [ - ('report_name', 1, 0, 'text', report_name), + ('report_name', COLS_TOT, 0, 'text', report_name), ] row_data = self.xls_row_template(c_specs, [x[0] for x in c_specs]) row_pos = self.xls_write_row( @@ -93,13 +116,13 @@ class general_ledger_xls(report_xls): cell_style = xlwt.easyxf(cell_format) cell_style_center = xlwt.easyxf(cell_format + _xs['center']) c_specs = [ - ('coa', 2, 0, 'text', _('Chart of Account')), - ('fy', 1, 0, 'text', _('Fiscal Year')), - ('df', 3, 0, 'text', _p.filter_form(data) == + ('coa', COLS_COA, 0, 'text', _('Chart of Account')), + ('fy', COLS_FY, 0, 'text', _('Fiscal Year')), + ('df', COLS_DF, 0, 'text', _p.filter_form(data) == 'filter_date' and _('Dates Filter') or _('Periods Filter')), - ('af', 1, 0, 'text', _('Accounts Filter')), - ('tm', 2, 0, 'text', _('Target Moves')), - ('ib', 2, 0, 'text', _('Initial Balance')), + ('af', COLS_AF, 0, 'text', _('Accounts Filter')), + ('tm', COLS_TM, 0, 'text', _('Target Moves')), + ('ib', COLS_IB, 0, 'text', _('Initial Balance')), ] row_data = self.xls_row_template(c_specs, [x[0] for x in c_specs]) @@ -110,8 +133,9 @@ class general_ledger_xls(report_xls): cell_style = xlwt.easyxf(cell_format) cell_style_center = xlwt.easyxf(cell_format + _xs['center']) c_specs = [ - ('coa', 2, 0, 'text', _p.chart_account.name), - ('fy', 1, 0, 'text', _p.fiscalyear.name if _p.fiscalyear else '-'), + ('coa', COLS_COA, 0, 'text', _p.chart_account.name), + ('fy', COLS_FY, 0, 'text', + _p.fiscalyear.name if _p.fiscalyear else '-'), ] df = _('From') + ': ' if _p.filter_form(data) == 'filter_date': @@ -124,11 +148,11 @@ class general_ledger_xls(report_xls): else: df += _p.stop_period.name if _p.stop_period else u'' c_specs += [ - ('df', 3, 0, 'text', df), - ('af', 1, 0, 'text', _p.accounts(data) and ', '.join( + ('df', COLS_DF, 0, 'text', df), + ('af', COLS_AF, 0, 'text', _p.accounts(data) and ', '.join( [account.code for account in _p.accounts(data)]) or _('All')), - ('tm', 2, 0, 'text', _p.display_target_move(data)), - ('ib', 2, 0, 'text', initial_balance_text[ + ('tm', COLS_TM, 0, 'text', _p.display_target_move(data)), + ('ib', COLS_IB, 0, 'text', initial_balance_text[ _p.initial_balance_mode]), ] row_data = self.xls_row_template(c_specs, [x[0] for x in c_specs]) @@ -161,6 +185,8 @@ class general_ledger_xls(report_xls): ('date', 1, 0, 'text', _('Date'), None, c_hdr_cell_style), ('period', 1, 0, 'text', _('Period'), None, c_hdr_cell_style), ('move', 1, 0, 'text', _('Entry'), None, c_hdr_cell_style), + ('supp_inv_no', 1, 0, 'text', _('Supplier Invoice Number'), + None, c_hdr_cell_style), ('journal', 1, 0, 'text', _('Journal'), None, c_hdr_cell_style), ('account_code', 1, 0, 'text', _('Account'), None, c_hdr_cell_style), @@ -212,7 +238,7 @@ class general_ledger_xls(report_xls): cumul_balance = 0.0 cumul_balance_curr = 0.0 c_specs = [ - ('acc_title', 11, 0, 'text', + ('acc_title', COLS_TOT, 0, 'text', ' - '.join([account.code, account.name])), ] row_data = self.xls_row_template( @@ -278,6 +304,8 @@ class general_ledger_xls(report_xls): ('period', 1, 0, 'text', line.get('period_code') or ''), ('move', 1, 0, 'text', line.get('move_name') or ''), + ('supp_inv_no', 1, 0, 'text', + line.get('supplier_invoice_number') or ''), ('journal', 1, 0, 'text', line.get('jcode') or ''), ('account_code', 1, 0, 'text', account.code), ('partner', 1, 0, 'text', @@ -316,7 +344,7 @@ class general_ledger_xls(report_xls): balance_credit = rowcol_to_cell(row_pos, 9) balance_formula = balance_debit + '-' + balance_credit c_specs = [ - ('acc_title', 7, 0, 'text', + ('acc_title', COLS_TOT - 4, 0, 'text', ' - '.join([account.code, account.name])), ('cum_bal', 1, 0, 'text', _('Cumulated Balance on Account'), @@ -344,4 +372,4 @@ class general_ledger_xls(report_xls): general_ledger_xls('report.account.account_report_general_ledger_xls', 'account.account', - parser=GeneralLedgerWebkit) + parser=GeneralLedgerWebkitSupplier)