pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[IMP] l10_fr_reports: carry over tax grid 27 to tax grid 22

Browse Source 
Using the new carryover feature, improve the french tax report to properly carryover the values from the tax grid 27 to the tax grid 22 of the next period.

Also slightly improve the carryover itself
to support total lines, and disallow to
carry over to lines from another report.

Task id #2452451

closes odoo/odoo#69887

Related: odoo/upgrade#2708
Related: odoo/enterprise#17953
Signed-off-by: Laurent Smet <smetl@users.noreply.github.com>
This commit is contained in:
Nicolas (vin)
2021-08-02 06:51:33 +00:00
parent 53d844c31a
commit 2bac8ffcea
4 changed files with 85 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/account/models/account_tax_carryover_line.py
+21 -11
View File
@@ -2,6 +2,7 @@ from odoo import api, fields, models, _
from odoo.exceptions import ValidationError
class AccountTaxCarryoverLine(models.Model):
_name = 'account.tax.carryover.line'
_description = 'Tax carryover line'
@@ -9,17 +10,26 @@ class AccountTaxCarryoverLine(models.Model):
name = fields.Char(required=True)
amount = fields.Float(required=True, default=0.0)
date = fields.Date(required=True, default=fields.Date.context_today, readonly=True)
tax_report_line_id = fields.Many2one(comodel_name='account.tax.report.line', string="Tax report line")
tax_report_country_id = fields.Many2one(related='tax_report_line_id.report_id.country_id')
company_id = fields.Many2one(comodel_name='res.company', string='Company',
required=True,
default=lambda self: self.env.company)
foreign_vat_fiscal_position_id = fields.Many2one(comodel_name='account.fiscal.position',
string='Fiscal position',
help="The foreign fiscal position for which this carryover is made.",
domain="[('company_id', '=', company_id), "
"('country_id', '=', tax_report_country_id), "
"('foreign_vat', '!=', False)]")
tax_report_line_id = fields.Many2one(
comodel_name='account.tax.report.line',
string="Tax report line",
domain="[('report_id', '=', tax_report_id)]",
)
tax_report_id = fields.Many2one(related='tax_report_line_id.report_id')
tax_report_country_id = fields.Many2one(related='tax_report_id.country_id')
company_id = fields.Many2one(
comodel_name='res.company', string='Company',
required=True,
default=lambda self: self.env.company,
)
foreign_vat_fiscal_position_id = fields.Many2one(
comodel_name='account.fiscal.position',
string="Fiscal position",
help="The foreign fiscal position for which this carryover is made.",
domain="[('company_id', '=', company_id), "
"('country_id', '=', tax_report_country_id), "
"('foreign_vat', '!=', False)]",
)
@api.constrains('foreign_vat_fiscal_position_id')
def _check_fiscal_position(self):
addons/account/models/account_tax_report.py
+53 -7
View File
@@ -1,6 +1,7 @@
# -*- coding: utf-8 -*-
from odoo import api, fields, models, _
from odoo.exceptions import ValidationError
from odoo.osv import expression
class AccountTaxReport(models.Model):
@@ -118,14 +119,17 @@ class AccountTaxReportLine(models.Model):
# The selection should be filled in localizations using the system
carry_over_condition_method = fields.Selection(
selection=[('no_negative_amount_carry_over_condition', 'No negative amount')],
string="Carryover method",
selection=[
('no_negative_amount_carry_over_condition', 'No negative amount'),
('always_carry_over_and_set_to_0', 'Always carry over and set to 0'),
],
string="Method",
help="The method used to determine if this line should be carried over."
)
carry_over_destination_line_id = fields.Many2one(
string="Carryover to",
string="Destination",
comodel_name="account.tax.report.line",
domain=[('tag_name', '!=', False)],
domain="[('report_id', '=', report_id)]",
help="The line to which the value of this line will be carried over to if needed."
" If left empty the line will carry over to itself."
)
@@ -134,6 +138,17 @@ class AccountTaxReportLine(models.Model):
comodel_name='account.tax.carryover.line',
inverse_name='tax_report_line_id',
)
is_carryover_persistent = fields.Boolean(
string="Persistent",
help="If set, each new carryover amount is added to the previous ones. And the amounts are carried over from period to period.\n"
"If not set, each new carryover amount overrides the former one.",
default=True,
)
is_carryover_used_in_balance = fields.Boolean(
string="Used in line balance",
help="If set, the carryover amount for this line will be used when calculating its balance in the report."
"This means that the carryover could affect other lines if they are using this one in their computation."
)
@api.model
def create(self, vals):
@@ -298,21 +313,28 @@ class AccountTaxReportLine(models.Model):
if neg_tags.name != '-'+record.tag_name or pos_tags.name != '+'+record.tag_name:
raise ValidationError(_("The tags linked to a tax report line should always match its tag name."))
def action_view_carryover_lines(self):
def action_view_carryover_lines(self, options):
''' Action when clicking on the "View carryover lines" in the carryover info popup.
Takes into account the report options, to get the correct lines depending on the current
company/companies.
:return: An action showing the account.tax.carryover.lines for the current tax report line.
'''
self.ensure_one()
target = self.carry_over_destination_line_id or self
domain = target._get_carryover_lines_domain(options)
carryover_lines = self.env['account.tax.carryover.line'].search(domain)
return {
'type': 'ir.actions.act_window',
'name': _('Carryover Lines For %s', self.name),
'name': _('Carryover Lines For %s', target.name),
'res_model': 'account.tax.carryover.line',
'view_type': 'list',
'view_mode': 'list',
'views': [[self.env.ref('account.account_tax_carryover_line_tree').id, 'list'],
[False, 'form']],
'domain': [('id', 'in', self.carryover_line_ids.ids)],
'domain': [('id', 'in', carryover_lines.ids)],
}
def _get_carryover_bounds(self, options, line_amount, carried_over_amount):
@@ -338,8 +360,32 @@ class AccountTaxReportLine(models.Model):
return None
def _get_carryover_lines_domain(self, options):
"""
:param options: The report options
:return: The domain that can be used to search for carryover lines for this tax report line.
Using this domain instead of directly accessing the lines ensure that we only pick the ones related to the
companies affecting the tax report.
"""
self.ensure_one()
domain = [('tax_report_line_id', '=', self.id)]
if options.get('multi_company'):
company_ids = [company['id'] for company in options['multi_company']]
domain = expression.AND([domain, [('company_id', 'in', company_ids)]])
else:
domain = expression.AND([domain, [('company_id', '=', self.env.company.id)]])
return domain
def no_negative_amount_carry_over_condition(self, options, line_amount, carried_over_amount):
# The bounds are (0, None).
# Lines below 0 will be set to 0 and reduce the balance of the carryover.
# Lines above 0 will never be carried over
return (0, None)
def always_carry_over_and_set_to_0(self, options, line_amount, carried_over_amount):
# The bounds are (0, 0).
# Lines below 0 will be set to 0 and reduce the balance of the carryover.
# Lines above 0 will be set to 0 and increase the balance of the carryover.
return (0, 0)
addons/account/views/account_tax_report_views.xml
+6 -2
View File
@@ -56,10 +56,11 @@
<tree>
<field name="company_id" invisible="1"/>
<field name="tax_report_country_id" invisible="1"/>
<field name="tax_report_id" invisible="1"/>
<field name="name"/>
<field name="date"/>
<field name="amount"/>
<field name="foreign_vat_fiscal_position_id"/>
<field name="foreign_vat_fiscal_position_id" optional="hide"/>
</tree>
</field>
</record>
@@ -72,6 +73,7 @@
<sheet>
<group>
<group>
<field name="tax_report_id" invisible="1"/>
<field name="name"/>
<field name="tax_report_line_id"/>
<field name="tax_report_country_id"/>
@@ -109,12 +111,14 @@
<field name="formula"/>
</group>
</group>
<group>
<group string="Carryover">
<group>
<field name="carry_over_condition_method" placeholder="No carryover"/>
<field name="is_carryover_persistent"/>
</group>
<group>
<field name="carry_over_destination_line_id" placeholder="Same report line"/>
<field name="is_carryover_used_in_balance"/>
</group>
</group>
<notebook>
addons/l10n_fr/data/tax_report_data.xml
+5 -1
View File
@@ -403,6 +403,7 @@
<field name="parent_id" ref="tax_report_tva_deductible"/>
<field name="report_id" ref="tax_report"/>
<field name="sequence">4</field>
<field name="is_carryover_used_in_balance">True</field>
</record>
<record id="tax_report_2C" model="account.tax.report.line">
@@ -486,6 +487,9 @@
<field name="report_id" ref="tax_report"/>
<field name="sequence">4</field>
<field name="formula">box_25 - box_26 - box_AA if box_25 - box_26 - box_AA > 0 else 0</field>
<field name="carry_over_condition_method">always_carry_over_and_set_to_0</field>
<field name="carry_over_destination_line_id" ref="tax_report_22"/>
<field name="is_carryover_persistent">False</field>
</record>
<record id="tax_report_taxes_a_payer" model="account.tax.report.line">
@@ -535,4 +539,4 @@
<field name="formula">box_28 + box_29</field>
</record>
</odoo>
</odoo>