intrastat_product: use shipping partner instead of invoicing partner …

…if any

Replace @api.onchange by compute methods with store=True, precompute=True and readonly=False for vat and region_code fields of computation lines

intrastat_product/models/intrastat_product_declaration.py

@@ -465,15 +465,16 @@ def _get_product_origin_country(self, inv_line, notedict):
             )
         return origin_country
 
-    def _get_vat(self, inv_line, notedict):
-        vat = False
+    def _get_partner_and_warn_vat(self, inv_line, notedict):
         inv = inv_line.move_id
-        if (
-            self.declaration_type == "dispatches"
-            and inv_line.move_id.fiscal_position_id.vat_required
-        ):
-            partner = inv.commercial_partner_id
-            vat = partner.vat
+        partner = (
+            inv.partner_shipping_id
+            and inv.partner_shipping_id.commercial_partner_id
+            or inv.commercial_partner_id
+        )
+        # Warnings about VAT
+        vat = partner.vat
+        if self.declaration_type == "dispatches":
             if vat:
                 if vat.startswith("GB"):
                     msg = _(
@@ -495,7 +496,7 @@ def _get_vat(self, inv_line, notedict):
                 notedict["partner"][partner.display_name][msg].add(
                     notedict["inv_origin"]
                 )
-        return vat
+        return partner
 
     def _update_computation_line_vals(self, inv_line, line_vals, notedict):
         """placeholder for localization modules"""
@@ -694,7 +695,7 @@ def _gather_invoices(self, notedict):
                 if not region_code:
                     region = self._get_region(inv_line, notedict)
 
-                vat = self._get_vat(inv_line, notedict)
+                partner = self._get_partner_and_warn_vat(inv_line, notedict)
 
                 line_vals = {
                     "parent_id": self.id,
@@ -710,8 +711,7 @@ def _gather_invoices(self, notedict):
                     "product_origin_country_id": product_origin_country.id or False,
                     "region_code": region_code,
                     "region_id": region and region.id or False,
-                    "vat": vat,
-                    "partner_id": invoice.commercial_partner_id.id,
+                    "partner_id": partner.id,
                 }
 
                 # extended declaration
@@ -1039,7 +1039,9 @@ class IntrastatProductComputationLine(models.Model):
     region_id = fields.Many2one("intrastat.region", string="Intrastat Region")
     # Note that, in l10n_fr_intrastat_product and maybe in other localization modules
     # region_id is left empty and Odoo writes directly in region_code
-    region_code = fields.Char()
+    region_code = fields.Char(
+        compute="_compute_region_code", store=True, readonly=False
+    )
     product_origin_country_id = fields.Many2one(
         "res.country",
         string="Country of Origin of the Product",
@@ -1054,16 +1056,21 @@ class IntrastatProductComputationLine(models.Model):
         help="2 letters ISO code of the country of origin of the product.\n"
         "Specify 'QU' when the country of origin is unknown.\n",
     )
-    vat = fields.Char(string="VAT Number")
+    vat = fields.Char(
+        compute="_compute_vat",
+        store=True,
+        readonly=False,
+        string="VAT Number",
+    )
 
     # extended declaration
     incoterm_id = fields.Many2one("account.incoterms", string="Incoterm")
     transport_id = fields.Many2one("intrastat.transport_mode", string="Transport Mode")
 
-    @api.onchange("region_id")
-    def _region_id_change(self):
-        if self.region_id:
-            self.region_code = self.region_id.code
+    @api.depends("region_id")
+    def _compute_region_code(self):
+        for this in self:
+            this.region_code = this.region_id and this.region_id.code or False
 
     @api.depends("src_dest_country_id")
     def _compute_src_dest_country_code(self):
@@ -1093,10 +1100,17 @@ def _check_vat(self):
             if this.vat and not is_valid(this.vat):
                 raise ValidationError(_("The VAT number '%s' is invalid.") % this.vat)
 
-    @api.onchange("partner_id")
-    def partner_id_change(self):
-        if self.partner_id and self.partner_id.vat:
-            self.vat = self.partner_id.vat
+    @api.depends("partner_id")
+    def _compute_vat(self):
+        for this in self:
+            vat = False
+            if (
+                this.partner_id
+                and this.partner_id.vat
+                and this.parent_id.declaration_type == "dispatches"
+            ):
+                vat = this.partner_id.vat
+            this.vat = vat
 
     def _group_line_hashcode_fields(self):
         self.ensure_one()