[MIG] shopinvader_base_url

Author: paradoxxxzero

shopinvader_base_url/__manifest__.py

@@ -8,7 +8,7 @@
     "version": "18.0.1.0.0",
     "category": "tools",
     "license": "AGPL-3",
-    "summary": "Keep history of url for products & categories  ",
+    "summary": "Keep history of url for products & categories",
     "author": "Akretion, ACSONE SA/NV",
     "website": "https://github.com/shopinvader/odoo-shopinvader-catalog",
     "depends": ["base", "base_sparse_field_list_support"],

shopinvader_base_url/models/abstract_url.py

@@ -83,7 +83,8 @@ def _compute_url_key(self):
     def _get_keyword_fields(self):
         """This method return a list of field that will be concatenated
         with '-' to generate the url
-        Ex: if you return ['name', 'code'] the url will be f"{record.name}-{record.code}"
+        Ex: if you return ['name', 'code'] the url will be
+        f"{record.name}-{record.code}"
 
         Note: the self already include in the context the lang of the record
         Note: you can return key like in depends ex: ["categ_id.name", "code"]
@@ -133,7 +134,7 @@ def _prepare_url(self, referential, lang, url_key):
             "res_model": self._name,
             "res_id": self.id,
             "referential": referential,
-            "lang_id": self.env["res.lang"]._lang_get_id(lang),
+            "lang_id": self.env["res.lang"]._lang_get(lang).id,
             "manual": False,
         }
 

shopinvader_base_url/models/url_url.py

@@ -19,21 +19,27 @@ class UrlUrl(models.Model):
     res_id = fields.Many2oneReference(
         string="Record ID",
         help="ID of the target record in the database",
-        model_field="res_model",
+        model_field="res_model_field",
         readonly=True,
         index=True,
     )
     res_model = fields.Selection(
         selection=lambda s: s._get_model_with_url_selection(), readonly=True, index=True
     )
+    res_model_field = fields.Char(
+        string="Model Name",
+        compute="_compute_res_model_field",
+        readonly=True,
+        store=True,
+    )
     redirect = fields.Boolean(help="If tick this url is a redirection to the new url")
     referential = fields.Selection(
         selection=lambda s: s._get_all_referential(),
         index=True,
         default="global",
         required=True,
     )
-    lang_id = fields.Many2one("res.lang", "Lang", index=True, required=True)
+    lang_id = fields.Many2one("res.lang", string="Lang", index=True, required=True)
     need_refresh = fields.Boolean()
 
     _sql_constraints = [
@@ -65,6 +71,11 @@ def _get_model_with_url_selection(self):
             )
         ]
 
+    @api.depends("res_model")
+    def _compute_res_model_field(self):
+        for record in self:
+            record.res_model_field = record.res_model
+
     def _get_all_referential(self):
         """Return the list of referential for your url, by default it's global
         but you can do your own implementation to have url per search engine

shopinvader_base_url/tests/test_abstract_url.py

@@ -8,32 +8,30 @@
 
 
 class TestAbstractUrl(TransactionCase, FakeModelLoader):
-    @classmethod
-    def setUpClass(cls):
-        super().setUpClass()
-        cls.loader = FakeModelLoader(cls.env, cls.__module__)
-        cls.loader.backup_registry()
+    def setUp(self):
+        super().setUp()
+        self.loader = FakeModelLoader(self.env, self.__module__)
+        self.loader.backup_registry()
         from .models import FakeCateg, FakeProduct
 
-        cls.loader.update_registry([FakeProduct, FakeCateg])
+        self.loader.update_registry([FakeProduct, FakeCateg])
 
-        cls.lang_en = cls.env.ref("base.lang_en")
-        cls.lang_fr = cls.env.ref("base.lang_fr")
-        cls.lang_fr.active = True
-        cls.product = (
-            cls.env["fake.product"]
+        self.lang_en = self.env.ref("base.lang_en")
+        self.lang_fr = self.env.ref("base.lang_fr")
+        self.lang_fr.active = True
+        self.product = (
+            self.env["fake.product"]
             .with_context(lang="en_US")
             .create({"name": "My Product"})
         )
-        cls.product.with_context(lang="fr_FR").name = "Mon Produit"
+        self.product.with_context(lang="fr_FR").name = "Mon Produit"
 
     def _expect_url_for_lang(self, lang, url_key):
         self.assertEqual(self.product._get_main_url("global", lang).key, url_key)
 
-    @classmethod
-    def tearDownClass(cls):
-        cls.loader.restore_registry()
-        super().tearDownClass()
+    def tearDown(self):
+        self.loader.restore_registry()
+        super().tearDown()
 
     def test_update_url_key(self):
         self.product._update_url_key("global", "en_US")
@@ -123,7 +121,8 @@ def test_write_inactive(self):
             mocked_redirect.assert_called_once()
 
     def test_update_twice_write_once(self):
-        """"
+        (
+            """"
         When we update twice the same record, the write method should be called
         only once. This is important because for example, by default, in
         shopinvader_search_engine_update, when the method write is called,
@@ -133,7 +132,9 @@ def test_update_twice_write_once(self):
         times the _update_url_key method. The first time, if every time the
         method make a write on the record, the record will end up with the
         state to_recompute.
-        """ ""
+        """
+            ""
+        )
 
         # we mock the write method to check the number of call but we want the
         # method to be executed

shopinvader_base_url/views/url_view.xml

@@ -12,6 +12,7 @@
                             name="res_model"
                             invisible="context.get('hide_res_model')"
                         />
+                        <field name="res_model_field" invisible="True" />
                         <field name="key" />
                         <field name="res_id" invisible="context.get('hide_res_id')" />
                         <field name="manual" />
@@ -30,6 +31,7 @@
                 <field name="lang_id" />
                 <field name="res_model" invisible="context.get('hide_res_model')" />
                 <field name="key" />
+                <field name="res_model_field" invisible="True" />
                 <field name="res_id" invisible="context.get('hide_res_id')" />
                 <field name="manual" />
                 <field name="redirect" />
@@ -38,14 +40,14 @@
     </record>
 
     <record id="base_url_action_view" model="ir.actions.act_window">
-        <field name="name">url list Viewer</field>
+        <field name="name">Url List</field>
         <field name="res_model">url.url</field>
         <field name="view_mode">list,form</field>
     </record>
     <!-- This Menu Item Must have a parent -->
     <menuitem
         id="url_menu_view"
-        name="Url_rewrite"
+        name="Url Rewrite"
         parent="base.next_id_9"
         action="base_url_action_view"
     />

test-requirements.txt

@@ -0,0 +1 @@
+odoo-test-helper