[IMP] create_recursive_abstract: when creating items via import of th…

…e model, allow to create dynamically parents

create_recursive_abstract/__manifest__.py

@@ -10,6 +10,6 @@
     "author": "GRAP",
     "website": "https://github.com/grap/grap-odoo-incubator",
     "license": "AGPL-3",
-    "depends": ["base"],
+    "depends": ["base_import"],
     "installable": True,
 }

create_recursive_abstract/models/__init__.py

@@ -1 +1,2 @@
 from . import create_recursive_mixin
+from . import base_import_import

create_recursive_abstract/models/base_import_import.py

@@ -0,0 +1,15 @@
+# Copyright (C) 2024 - Today: GRAP (http://www.grap.coop)
+# @author: Sylvain LE GAL (https://twitter.com/legalsylvain)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
+from odoo import models
+
+
+class BaseImportImport(models.TransientModel):
+    _inherit = "base_import.import"
+
+    def execute_import(self, fields, columns, options, dryrun=False):
+        self.ensure_one()
+        return super(
+            BaseImportImport, self.with_context(imported_model=self.res_model)
+        ).execute_import(fields, columns, options, dryrun=dryrun)

create_recursive_abstract/models/create_recursive_mixin.py

@@ -35,22 +35,35 @@ def _name_create(self, name):
         - then, create an item named 'Child' with the parent found as parent_id.
         """
         vals = {}
+        self._create_recursive_alter_vals(vals, name)
+
+        item = self.create(vals)
+        return item.name_get()[0]
+
+    @api.model
+    def _create_recursive_alter_vals(self, vals, name=False):
+        name = name and name or vals.get("name", "")
         if "/" in name:
             splitted_name = name.split("/")
             parent_name = (" / ".join([x.strip() for x in splitted_name[:-1]])).strip()
             item_name = splitted_name[-1:][0].strip()
             parent_id = self._create_recursive_get_or_create_parent_id(parent_name)
-            vals = {"name": item_name, "parent_id": parent_id}
+            vals.update({"name": item_name, "parent_id": parent_id})
         else:
-            vals = {"name": name}
-        item = self.create(vals)
-        return item.name_get()[0]
+            vals.update({"name": name})
 
     @api.model_create_multi
     def create(self, vals_list):
-        for value in vals_list:
-            if "name" in value:
-                value["name"] = value["name"].replace("/", "-").strip()
+        if self.env.context.get("imported_model") == self._name:
+            # Creation from import of model
+            # create parents if doesn't exist
+            for vals in vals_list:
+                self._create_recursive_alter_vals(vals)
+        else:
+            # Regular creation, removing bad "/"
+            for vals in vals_list:
+                if "name" in vals:
+                    vals["name"] = vals["name"].replace("/", "-").strip()
         return super().create(vals_list)
 
     def write(self, vals):

create_recursive_abstract/tests/test_create_recursive_abstract.py

@@ -44,3 +44,11 @@ def test_name_create(self):
         categ = self.model.browse(res[0])
         self.assertEqual(categ.name, "New Category")
         self.assertEqual(categ.parent_id, self.child_item)
+
+    def test_create_via_import(self):
+        categ = self.model.with_context(imported_model=self.model._name).create(
+            {"name": "Parent / Child"}
+        )
+        self.assertEqual(categ.name, "Child")
+        self.assertTrue(categ.parent_id)
+        self.assertEqual(categ.parent_id.name, "Parent")