[MIG] l10n_it_intrastat_statement: Migration to 14.0

Author: fredzamoabg

l10n_it_intrastat_statement/README.rst

@@ -14,13 +14,13 @@ ITA - Dichiarazione Intrastat
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fl10n--italy-lightgray.png?logo=github
-    :target: https://github.com/OCA/l10n-italy/tree/12.0/l10n_it_intrastat_statement
+    :target: https://github.com/OCA/l10n-italy/tree/14.0/l10n_it_intrastat_statement
     :alt: OCA/l10n-italy
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_intrastat_statement
+    :target: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_intrastat_statement
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png
-    :target: https://runbot.odoo-community.org/runbot/122/12.0
+    :target: https://runbot.odoo-community.org/runbot/122/14.0
     :alt: Try me on Runbot
 
 |badge1| |badge2| |badge3| |badge4| |badge5| 
@@ -82,7 +82,7 @@ I dati presi dalle fatture e dalle note di credito indicate come soggette ad Int
 
 N.B.: i record presenti nelle schede "Rettifica beni - Sezione 2" e "Rettifica servizi - Sezione 4", sia per gli acquisti che per le vendite, vanno modificati per inserire i dati obbligatori mancanti.
 
-Inseriti i dati e salvata la dichiarazione, è possibile procedere all’elaborazione dei file da inviare all’Agenzia delle Dogane tramite l’apposito pulsante «Esporta file». 
+Inseriti i dati e salvata la dichiarazione, è possibile procedere all’elaborazione dei file da inviare all’Agenzia delle Dogane tramite l’apposito pulsante «Esporta file».
 
 Il pulsante fa partire una procedura guidata, che permette di scegliere quale tipo di file estrarre:
 
@@ -100,7 +100,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/l10n-italy/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us smashing it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/l10n-italy/issues/new?body=module:%20l10n_it_intrastat_statement%0Aversion:%2012.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/l10n-italy/issues/new?body=module:%20l10n_it_intrastat_statement%0Aversion:%2014.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -136,6 +136,6 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
-This module is part of the `OCA/l10n-italy <https://github.com/OCA/l10n-italy/tree/12.0/l10n_it_intrastat_statement>`_ project on GitHub.
+This module is part of the `OCA/l10n-italy <https://github.com/OCA/l10n-italy/tree/14.0/l10n_it_intrastat_statement>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

l10n_it_intrastat_statement/__manifest__.py

@@ -3,12 +3,12 @@
 
 {
     "name": "ITA - Dichiarazione Intrastat",
-    "version": "12.0.1.2.2",
+    "version": "14.0.1.0.0",
     "category": "Account",
     "summary": "Dichiarazione Intrastat per l'Agenzia delle Dogane",
     "author": "Openforce, Link IT srl, Agile Business Group, "
     "Odoo Community Association (OCA)",
-    "website": "https://github.com/OCA/l10n-italy" "12.0/l10n_it_intrastat_statement",
+    "website": "https://github.com/OCA/l10n-italy",
     "license": "AGPL-3",
     "depends": [
         "l10n_it_intrastat",

l10n_it_intrastat_statement/migration/14.0.1.0.0/noupdate_changes.xml

@@ -0,0 +1,8 @@
+<?xml version='1.0' encoding='utf-8' ?>
+<odoo>
+    <record id="statement_multi_company" model="ir.rule">
+        <field
+            name="domain_force"
+        >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+    </record>
+</odoo>

l10n_it_intrastat_statement/migration/14.0.1.0.0/post-migration.py

@@ -0,0 +1,10 @@
+from openupgradelib import openupgrade
+
+
+@openupgrade.migrate()
+def migrate(env, version):
+    openupgrade.load_data(
+        env.cr,
+        "l10n_it_intrastat_statement",
+        "migrations/14.0.1.0.0/noupdate_changes.xml",
+    )

l10n_it_intrastat_statement/models/intrastat_statement.py

@@ -45,7 +45,6 @@ class AccountIntrastatStatement(models.Model):
     _description = "Intrastat Statement"
     _rec_name = "number"
 
-    @api.multi
     def round_min_amount(self, amount, company=None, prec_digits=None, truncate=False):
         """
         Return an integer representing `amount`,
@@ -76,7 +75,6 @@ def round_min_amount(self, amount, company=None, prec_digits=None, truncate=Fals
 
         return max(round_amount or 1, company.intrastat_min_amount)
 
-    @api.multi
     def _compute_amount_section(self, section_type, section_number):
         """
         Compute operation_number and amount for specified section.
@@ -149,7 +147,6 @@ def _compute_progressive(self):
         st = self.search([], order="number desc", limit=1)
         return (st.number or 0) + 1
 
-    @api.multi
     def recompute_sequence_lines(self):
         for statement in self:
             for section_type in ["purchase", "sale"]:
@@ -174,23 +171,23 @@ def _get_sequence(self):
     company_id = fields.Many2one(
         comodel_name="res.company",
         string="Company",
-        default=lambda self: self.env.user.company_id.id,
+        default=lambda self: self.env.company.id,
         required=True,
     )
     vat_taxpayer = fields.Char(
         string="Taxpayer VAT Number",
         required=True,
-        default=lambda self: self.env.user.company_id.partner_id.vat
-        and self.env.user.company_id.partner_id.vat[2:]
+        default=lambda self: self.env.company.partner_id.vat
+        and self.env.company.partner_id.vat[2:]
         or False,
     )
     intrastat_vat_delegate = fields.Char(
         string="Delegate VAT Number",
-        default=lambda self: self.env.user.company_id.intrastat_delegated_vat,
+        default=lambda self: self.env.company.intrastat_delegated_vat,
     )
     intrastat_name_delegate = fields.Char(
         string="Delegate Name",
-        default=lambda self: self.env.user.company_id.intrastat_delegated_name,
+        default=lambda self: self.env.company.intrastat_delegated_name,
     )
     fiscalyear = fields.Integer(
         string="Year", required=True, default=fields.Date.today().year
@@ -244,7 +241,7 @@ def _get_sequence(self):
         comodel_name="account.intrastat.custom",
         string="Customs Section",
         required=True,
-        default=lambda self: self.env.user.company_id.intrastat_custom_id,
+        default=lambda self: self.env.company.intrastat_custom_id,
     )
     sale = fields.Boolean(string="Sales", default=True)
     purchase = fields.Boolean(string="Purchases", default=True)
@@ -415,14 +412,12 @@ def create(self, vals):
         statement._normalize_statement()
         return statement
 
-    @api.multi
     def write(self, vals):
         res = super().write(vals)
         self._normalize_statement()
         self.recompute_sequence_lines()
         return res
 
-    @api.multi
     @api.depends("fiscalyear", "period_type", "period_number")
     def _compute_dates(self):
         for statement in self:
@@ -440,7 +435,6 @@ def _compute_dates(self):
             statement.date_start = fields.Date.to_date(period_date_start)
             statement.date_stop = fields.Date.to_date(period_date_stop)
 
-    @api.multi
     def get_dates_start_stop(self):
         self.ensure_one()
         year = self.fiscalyear
@@ -462,7 +456,6 @@ def get_dates_start_stop(self):
             )
         return period_date_start, period_date_stop
 
-    @api.multi
     def _get_period_ref(self):
         self.ensure_one()
         res = {"year_id": self.fiscalyear}
@@ -474,7 +467,6 @@ def _get_period_ref(self):
             res.update({"month": self.period_number})
         return res
 
-    @api.multi
     def _normalize_statement(self):
         # Unlink lines sale/purchase sections
         self.ensure_one()
@@ -484,7 +476,6 @@ def _normalize_statement(self):
             self._unlink_sections(section_type="purchase")
         return True
 
-    @api.multi
     def _unlink_sections(self, section_type="all"):
         """
         Unlink lines sale/purchase sections.
@@ -506,7 +497,6 @@ def _unlink_sections(self, section_type="all"):
             self.purchase_section4_ids.unlink()
         return True
 
-    @api.multi
     def _get_progressive_interchange(self):
         self.ensure_one()
         prg = 0
@@ -536,7 +526,6 @@ def _get_file_name(self):
                 "{:2s}".format(str(prg).zfill(2)),
             )
 
-    @api.multi
     def _prepare_export_head(self):
         self.ensure_one()
 
@@ -596,7 +585,6 @@ def _prepare_export_head(self):
         rcd += "\r\n"
         return rcd
 
-    @api.multi
     def _prepare_export_prefix(self, ref_number, line=None):
         """
         Fixed part for every line of the exported file
@@ -651,7 +639,6 @@ def _format_negative_number_frontispiece(self, number):
         number[len(number) - 1] = interchange[int(last_char)]
         return "".join(number)
 
-    @api.multi
     def _prepare_export_frontispiece(self, kind, ref_number):
         self.ensure_one()
         rcd = self._prepare_export_prefix(ref_number)
@@ -695,7 +682,6 @@ def _prepare_export_frontispiece(self, kind, ref_number):
         rcd += "\r\n"
         return rcd
 
-    @api.multi
     def generate_file_export(self):
         self.ensure_one()
         file_content = ""
@@ -765,7 +751,6 @@ def generate_file_export(self):
 
         return file_content
 
-    @api.multi
     def compute_statement(self):
         self.ensure_one()
         # Unlink existing lines
@@ -776,19 +761,19 @@ def compute_statement(self):
 
         # Search intrastat lines
         domain = [
-            ("move_id.date", ">=", period_date_start),
-            ("move_id.date", "<=", period_date_stop),
+            ("invoice_date", ">=", period_date_start),
+            ("invoice_date", "<=", period_date_stop),
             ("intrastat", "=", True),
         ]
         inv_type = []
         if self.sale:
             inv_type += ["out_invoice", "out_refund"]
         if self.purchase:
             inv_type += ["in_invoice", "in_refund"]
-        domain.append(("type", "in", inv_type))
+        domain.append(("move_type", "in", inv_type))
 
         statement_data = dict()
-        invoices = self.env["account.invoice"].search(domain)
+        invoices = self.env["account.move"].search(domain)
         for inv_intra_line in invoices.mapped("intrastat_line_ids"):
             for section_type in ["purchase", "sale"]:
                 for section_number in range(1, 5):
@@ -844,7 +829,6 @@ def get_section_model(section_type, section_number):
     def get_section_field_name(section_type, section_number):
         return "{}_section{}_ids".format(section_type, section_number)
 
-    @api.multi
     def refund_line(self, line, to_ref_obj):
         """Refund line into sale if period ref is the same of the statement"""
         self.ensure_one()