From ecabbbb6e6df5ac5581541a4a4a2eeb2c1c76dc0 Mon Sep 17 00:00:00 2001 From: Quentin Groulard Date: Thu, 28 Mar 2019 16:51:50 +0100 Subject: [PATCH] [FIX] Few fixes to solve warnings: - _description in account_financial_report_abstract - _description in vat.report.wizard - Two fields using same label 'Accounts' in account.group - Two fields using same label 'Account' in report_journal_ledger_move_line - Two fields using same label 'Partner' in report_journal_ledger_move_line - Two fields using same label 'Accounts' in report_trial_balance_account - Two fields using same label 'Tax' in report_vat_report_taxtag - Two fields using same label 'Tax' in report_vat_report_tax - Two fields using same label 'Filter accounts' in general.ledger.report.wizard --- account_financial_report/models/account_group.py | 2 +- account_financial_report/report/abstract_report.py | 1 + account_financial_report/report/journal_ledger.py | 4 +++- account_financial_report/report/trial_balance.py | 4 ++-- account_financial_report/report/vat_report.py | 6 ++++-- account_financial_report/wizard/general_ledger_wizard.py | 2 +- account_financial_report/wizard/vat_report_wizard.py | 1 + 7 files changed, 13 insertions(+), 7 deletions(-) diff --git a/account_financial_report/models/account_group.py b/account_financial_report/models/account_group.py index 0747f74a..bbc3b0b3 100644 --- a/account_financial_report/models/account_group.py +++ b/account_financial_report/models/account_group.py @@ -22,7 +22,7 @@ class AccountGroup(models.Model): compute_account_ids = fields.Many2many( 'account.account', compute='_compute_group_accounts', - string="Accounts", store=True) + string="Compute accounts", store=True) @api.multi @api.depends('parent_id', 'parent_id.level') diff --git a/account_financial_report/report/abstract_report.py b/account_financial_report/report/abstract_report.py index 4094af1c..7fe0c54e 100644 --- a/account_financial_report/report/abstract_report.py +++ b/account_financial_report/report/abstract_report.py @@ -6,6 +6,7 @@ from odoo import models class AbstractReport(models.AbstractModel): _name = 'account_financial_report_abstract' + _description = 'Abstract Report' def _transient_clean_rows_older_than(self, seconds): assert self._transient, \ diff --git a/account_financial_report/report/journal_ledger.py b/account_financial_report/report/journal_ledger.py index 352b4c49..8688375b 100644 --- a/account_financial_report/report/journal_ledger.py +++ b/account_financial_report/report/journal_ledger.py @@ -715,7 +715,8 @@ class ReportJournalLedgerMoveLine(models.TransientModel): ondelete='cascade', ) account_id = fields.Many2one( - comodel_name='account.account' + comodel_name='account.account', + string='Account ID', ) account = fields.Char() account_code = fields.Char() @@ -723,6 +724,7 @@ class ReportJournalLedgerMoveLine(models.TransientModel): partner = fields.Char() partner_id = fields.Many2one( comodel_name='res.partner', + string='Partner ID', ) date = fields.Date() entry = fields.Char() diff --git a/account_financial_report/report/trial_balance.py b/account_financial_report/report/trial_balance.py index c668127a..5f5c8418 100644 --- a/account_financial_report/report/trial_balance.py +++ b/account_financial_report/report/trial_balance.py @@ -93,10 +93,10 @@ class TrialBalanceReportAccount(models.TransientModel): index=True ) child_account_ids = fields.Char( - string="Accounts") + string="Child accounts") compute_account_ids = fields.Many2many( 'account.account', - string="Accounts", store=True) + string="Compute accounts", store=True) # Data fields, used for report display code = fields.Char() diff --git a/account_financial_report/report/vat_report.py b/account_financial_report/report/vat_report.py index 6dea150e..795dd1bf 100644 --- a/account_financial_report/report/vat_report.py +++ b/account_financial_report/report/vat_report.py @@ -64,7 +64,8 @@ class VATReportTaxTags(models.TransientModel): # Data fields, used to browse report data tax_ids = fields.One2many( comodel_name='report_vat_report_tax', - inverse_name='report_tax_id' + inverse_name='report_tax_id', + string='Taxes' ) @@ -82,7 +83,8 @@ class VATReportTax(models.TransientModel): # Data fields, used to keep link with real object tax_id = fields.Many2one( 'account.tax', - index=True + index=True, + string='Tax ID', ) # Data fields, used for report display diff --git a/account_financial_report/wizard/general_ledger_wizard.py b/account_financial_report/wizard/general_ledger_wizard.py index 621cc070..de119dac 100644 --- a/account_financial_report/wizard/general_ledger_wizard.py +++ b/account_financial_report/wizard/general_ledger_wizard.py @@ -65,7 +65,7 @@ class GeneralLedgerReportWizard(models.TransientModel): ) analytic_tag_ids = fields.Many2many( comodel_name='account.analytic.tag', - string='Filter accounts', + string='Filter analytic tags', ) account_journal_ids = fields.Many2many( comodel_name='account.journal', diff --git a/account_financial_report/wizard/vat_report_wizard.py b/account_financial_report/wizard/vat_report_wizard.py index 2f78525b..2b61e2a3 100644 --- a/account_financial_report/wizard/vat_report_wizard.py +++ b/account_financial_report/wizard/vat_report_wizard.py @@ -9,6 +9,7 @@ from odoo.exceptions import ValidationError class VATReportWizard(models.TransientModel): _name = "vat.report.wizard" + _description = "VAT Report Wizard" company_id = fields.Many2one( comodel_name='res.company',