From b4a7d748c57dd10700e1a05a0db847f334b7e42a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?V=C3=ADctor=20Mart=C3=ADnez?=
 <victor.martinez@tecnativa.com>
Date: Thu, 25 Feb 2021 09:49:39 +0100
Subject: [PATCH] [FIX] account_asset_management: Fix onchange account_id and
 asset_profile_id to prevent account_id is empty in some use cases

---
 .../models/account_move.py                    |  7 +++---
 .../readme/CONTRIBUTORS.rst                   |  1 +
 .../tests/test_account_asset_management.py    | 25 +++++++++++++++++++
 3 files changed, 30 insertions(+), 3 deletions(-)

diff --git a/account_asset_management/models/account_move.py b/account_asset_management/models/account_move.py
index 4b1f0f9b8..62e8158a1 100644
--- a/account_asset_management/models/account_move.py
+++ b/account_asset_management/models/account_move.py
@@ -151,13 +151,14 @@ class AccountMoveLine(models.Model):
 
     @api.onchange("account_id")
     def _onchange_account_id(self):
-        self.asset_profile_id = self.account_id.asset_profile_id
+        if self.account_id.asset_profile_id:
+            self.asset_profile_id = self.account_id.asset_profile_id
         super()._onchange_account_id()
 
     @api.onchange("asset_profile_id")
     def _onchange_asset_profile_id(self):
-        self.account_id = self.asset_profile_id.account_asset_id
-        super()._onchange_account_id()
+        if self.asset_profile_id.account_asset_id:
+            self.account_id = self.asset_profile_id.account_asset_id
 
     @api.model_create_multi
     def create(self, vals_list):
diff --git a/account_asset_management/readme/CONTRIBUTORS.rst b/account_asset_management/readme/CONTRIBUTORS.rst
index 4504972bc..ccabe05d1 100644
--- a/account_asset_management/readme/CONTRIBUTORS.rst
+++ b/account_asset_management/readme/CONTRIBUTORS.rst
@@ -13,3 +13,4 @@
 
   * Ernesto Tejeda
   * Pedro M. Baeza
+  * Víctor Martínez
diff --git a/account_asset_management/tests/test_account_asset_management.py b/account_asset_management/tests/test_account_asset_management.py
index a64772bb3..59fe5ee00 100644
--- a/account_asset_management/tests/test_account_asset_management.py
+++ b/account_asset_management/tests/test_account_asset_management.py
@@ -89,6 +89,31 @@ class TestAssetManagement(SavepointCase):
             line_form.quantity = 1
         cls.invoice_2 = move_form.save()
 
+    def test_invoice_line_without_product(self):
+        tax = self.env["account.tax"].create(
+            {
+                "name": "TAX 15%",
+                "amount_type": "percent",
+                "type_tax_use": "purchase",
+                "amount": 15.0,
+            }
+        )
+        move_form = Form(
+            self.env["account.move"].with_context(
+                default_type="in_invoice", check_move_validity=False
+            )
+        )
+        move_form.partner_id = self.partner
+        move_form.journal_id = self.journal
+        with move_form.invoice_line_ids.new() as line_form:
+            line_form.name = "Line 1"
+            line_form.price_unit = 200.0
+            line_form.quantity = 1
+            line_form.tax_ids.clear()
+            line_form.tax_ids.add(tax)
+        invoice = move_form.save()
+        self.assertEqual(invoice.partner_id, self.partner)
+
     def test_01_nonprorata_basic(self):
         """Basic tests of depreciation board computations and postings."""
         #