[FIX] Access error on ir.attachment create

As normal user only have read access to delay.export, we need sudo to chreate an attachment on this record Use user 6 (Marc Demo) in the tests, not admin (2)
OCA · Oct 31, 2023 · 47b087d · 47b087d
1 parent 55ad859
commit 47b087d
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 10 deletions.
diff --git a/base_export_async/models/delay_export.py b/base_export_async/models/delay_export.py
@@ -90,14 +90,18 @@ def export(self, params):
         export_record = self.sudo().create({"user_ids": [(6, 0, users.ids)]})
 
         name = "{}.{}".format(model_name, export_format)
-        attachment = self.env["ir.attachment"].create(
-            {
-                "name": name,
-                "datas": base64.b64encode(content),
-                "type": "binary",
-                "res_model": self._name,
-                "res_id": export_record.id,
-            }
+        attachment = (
+            self.env["ir.attachment"]
+            .sudo()
+            .create(
+                {
+                    "name": name,
+                    "datas": base64.b64encode(content),
+                    "type": "binary",
+                    "res_model": self._name,
+                    "res_id": export_record.id,
+                }
+            )
         )
 
         url = "{}/web/content/ir.attachment/{}/datas/{}?download=true".format(

diff --git a/base_export_async/tests/test_base_export_async.py b/base_export_async/tests/test_base_export_async.py
@@ -21,7 +21,7 @@
             "domain": [],
             "context": {"lang": "en_US", "tz": "Europe/Brussels", "uid": 2},
             "import_compat": false,
-            "user_ids": [2]
+            "user_ids": [6]
             }"""
 }
 
@@ -35,7 +35,7 @@
             "domain": [],
             "context": {"lang": "en_US", "tz": "Europe/Brussels", "uid": 2},
             "import_compat": false,
-            "user_ids": [2]
+            "user_ids": [6]
             }"""
 }