From d1d43236c61769c70a4c279316cf286146188665 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Duy=20=28=C4=90=E1=BB=97=20Anh=29?=
 <doanhduyxavie@gmail.com>
Date: Mon, 21 Oct 2024 11:01:13 +0700
Subject: [PATCH] [IMG] slugify deprecation

---
 .../migration_scripts/migrate_170_180.py      | 33 +++++++++++++++++++
 tests/data_result/module_170_180/__init__.py  |  1 +
 .../module_170_180/controllers/__init__.py    |  1 +
 .../module_170_180/controllers/main.py        |  9 +++++
 .../module_170_180/models/__init__.py         |  1 +
 .../module_170_180/models/website.py          |  9 +++++
 tests/data_template/module_170/__init__.py    |  1 +
 .../module_170/controllers/__init__.py        |  1 +
 .../module_170/controllers/main.py            | 10 ++++++
 .../module_170/models/__init__.py             |  1 +
 .../module_170/models/website.py              | 10 ++++++
 11 files changed, 77 insertions(+)
 create mode 100644 tests/data_result/module_170_180/controllers/__init__.py
 create mode 100644 tests/data_result/module_170_180/controllers/main.py
 create mode 100644 tests/data_result/module_170_180/models/website.py
 create mode 100644 tests/data_template/module_170/controllers/__init__.py
 create mode 100644 tests/data_template/module_170/controllers/main.py
 create mode 100644 tests/data_template/module_170/models/website.py

diff --git a/odoo_module_migrate/migration_scripts/migrate_170_180.py b/odoo_module_migrate/migration_scripts/migrate_170_180.py
index 8734fa08..df0d4c97 100644
--- a/odoo_module_migrate/migration_scripts/migrate_170_180.py
+++ b/odoo_module_migrate/migration_scripts/migrate_170_180.py
@@ -100,9 +100,42 @@ def replace_user_has_groups(
             logger.error(f"Error processing file {file}: {str(e)}")
 
 
+def replace_slugify(
+    logger, module_path, module_name, manifest_path, migration_steps, tools
+):
+    files_to_process = tools.get_files(module_path, (".py"))
+
+    for file in files_to_process:
+        try:
+            content = tools._read_content(file)
+            content = re.sub(
+                r"from\s+odoo\.addons\.http_routing\.models\.ir_http\s+import\s+slugify\b.*\n",
+                "",
+                content,
+            )
+            # process in controller (*.py) file are using request
+            has_request = "request" in content
+            if has_request:
+                content = re.sub(
+                    r"""(?<!request\.)\bslugify\(([^)]+)\)""",
+                    r"""request.env["ir.http"]._slugify(\1)""",
+                    content,
+                )
+            else:
+                content = re.sub(
+                    r"""\bslugify\(([^)]+)\)""",
+                    r"""self.env["ir.http"]._slugify(\1)""",
+                    content,
+                )
+            tools._write_content(file, content)
+        except Exception as e:
+            logger.error(f"Error processing file {file}: {str(e)}")
+
+
 class MigrationScript(BaseMigrationScript):
     _GLOBAL_FUNCTIONS = [
         replace_tree_with_list_in_views,
         replace_chatter_blocks,
         replace_user_has_groups,
+        replace_slugify,
     ]
diff --git a/tests/data_result/module_170_180/__init__.py b/tests/data_result/module_170_180/__init__.py
index 0650744f..f7209b17 100644
--- a/tests/data_result/module_170_180/__init__.py
+++ b/tests/data_result/module_170_180/__init__.py
@@ -1 +1,2 @@
 from . import models
+from . import controllers
diff --git a/tests/data_result/module_170_180/controllers/__init__.py b/tests/data_result/module_170_180/controllers/__init__.py
new file mode 100644
index 00000000..12a7e529
--- /dev/null
+++ b/tests/data_result/module_170_180/controllers/__init__.py
@@ -0,0 +1 @@
+from . import main
diff --git a/tests/data_result/module_170_180/controllers/main.py b/tests/data_result/module_170_180/controllers/main.py
new file mode 100644
index 00000000..19240bd9
--- /dev/null
+++ b/tests/data_result/module_170_180/controllers/main.py
@@ -0,0 +1,9 @@
+from odoo import http
+from odoo.http import request
+
+
+class MainController(http.Controller):
+    @http.route("/home/main", type="http", auth="public", website=True)
+    def redirect_to_main(self):
+        partner_name = request.env["ir.http"]._slugify(request.env.user.partner_id.name)
+        pass
diff --git a/tests/data_result/module_170_180/models/__init__.py b/tests/data_result/module_170_180/models/__init__.py
index 91fed54d..2758140f 100644
--- a/tests/data_result/module_170_180/models/__init__.py
+++ b/tests/data_result/module_170_180/models/__init__.py
@@ -1 +1,2 @@
 from . import res_partner
+from . import website
diff --git a/tests/data_result/module_170_180/models/website.py b/tests/data_result/module_170_180/models/website.py
new file mode 100644
index 00000000..8b76e566
--- /dev/null
+++ b/tests/data_result/module_170_180/models/website.py
@@ -0,0 +1,9 @@
+from odoo import api, models
+
+
+class Website(models.Model):
+    _inherit = "website"
+
+    @api.model
+    def example_method_use_slugify(self, page_name):
+        return "/" + self.env["ir.http"]._slugify(page_name, max_length=1024, path=True)
diff --git a/tests/data_template/module_170/__init__.py b/tests/data_template/module_170/__init__.py
index 0650744f..f7209b17 100644
--- a/tests/data_template/module_170/__init__.py
+++ b/tests/data_template/module_170/__init__.py
@@ -1 +1,2 @@
 from . import models
+from . import controllers
diff --git a/tests/data_template/module_170/controllers/__init__.py b/tests/data_template/module_170/controllers/__init__.py
new file mode 100644
index 00000000..12a7e529
--- /dev/null
+++ b/tests/data_template/module_170/controllers/__init__.py
@@ -0,0 +1 @@
+from . import main
diff --git a/tests/data_template/module_170/controllers/main.py b/tests/data_template/module_170/controllers/main.py
new file mode 100644
index 00000000..903ff2fe
--- /dev/null
+++ b/tests/data_template/module_170/controllers/main.py
@@ -0,0 +1,10 @@
+from odoo import http
+from odoo.addons.http_routing.models.ir_http import slugify
+from odoo.http import request
+
+
+class MainController(http.Controller):
+    @http.route("/home/main", type="http", auth="public", website=True)
+    def redirect_to_main(self):
+        partner_name = slugify(request.env.user.partner_id.name)
+        pass
diff --git a/tests/data_template/module_170/models/__init__.py b/tests/data_template/module_170/models/__init__.py
index 91fed54d..2758140f 100644
--- a/tests/data_template/module_170/models/__init__.py
+++ b/tests/data_template/module_170/models/__init__.py
@@ -1 +1,2 @@
 from . import res_partner
+from . import website
diff --git a/tests/data_template/module_170/models/website.py b/tests/data_template/module_170/models/website.py
new file mode 100644
index 00000000..a95b4163
--- /dev/null
+++ b/tests/data_template/module_170/models/website.py
@@ -0,0 +1,10 @@
+from odoo import api, models
+from odoo.addons.http_routing.models.ir_http import slugify
+
+
+class Website(models.Model):
+    _inherit = "website"
+
+    @api.model
+    def example_method_use_slugify(self, page_name):
+        return "/" + slugify(page_name, max_length=1024, path=True)