odoo-addon-dms 16.0.1.8.0.3__py3-none-any.whl → 17.0.1.0.0.2__py3-none-any.whl

odoo/addons/dms/{views/dms_portal_templates.xml → template/portal.xml}

@@ -1,3 +1,8 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<!--
+    Copyright 2024 Subteno - Timothée Vannier (https://www.subteno.com).
+    License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
+-->
 <odoo>
     <template
         id="portal_my_home_dms"
@@ -5,10 +10,15 @@
         inherit_id="portal.portal_my_home"
         priority="20"
     >
-        <xpath expr="//div[hasclass('o_portal_docs')]" position="inside">
+        <xpath expr="//div[hasclass('o_portal_docs')]" position="before">
+            <t t-set="portal_client_category_enable" t-value="True" />
+        </xpath>
+        <xpath expr="//div[@id='portal_client_category']" position="inside">
             <t t-call="portal.portal_docs_entry">
                 <t t-set="title">Documents</t>
+                <t t-set="text" t-value="'Manage your documents'" />
                 <t t-set="url" t-value="'/my/dms'" />
+                <t t-set="icon" t-value="'/dms/static/description/portal_icon.svg'" />
                 <t t-set="placeholder_count" t-value="'dms_directory_count'" />
             </t>
         </xpath>
@@ -101,9 +111,9 @@
                                     >
                                         <img
                                             class="o_portal_contact_img"
-                                            t-att-src="dms_file.icon_url"
+                                            t-att-src="dms_file.icon_url + (('&amp;access_token=' + access_token) if access_token else '')"
                                         />
-                                        <span t-esc="dms_file.name" />
+                                            <span t-esc="dms_file.name" />
                                     </a>
                                 </td>
                                 <td>

odoo/addons/dms/tests/common.py

@@ -1,6 +1,7 @@
 # Copyright 2017-2019 MuK IT GmbH.
 # Copyright 2020 Creu Blanca
 # Copyright 2021-2024 Tecnativa - Víctor Martínez
+# Copyright 2024 Subteno - Timothée Vannier (https://www.subteno.com).
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
 import base64
 import functools
@@ -173,23 +174,38 @@ class StorageAttachmentBaseCase(DocumentsBaseCase):
         cls.partner_model_id = cls.env.ref("base.model_res_partner")
         cls.create_directory(storage=cls.storage, model_id=cls.partner_model_id)
         cls.partner = cls.partner_model.create({"name": "test partner"})
+        cls.other_partner = cls.partner_model.create({"name": "other partner"})
         cls.model_partner = cls.env.ref("base.model_res_partner")
 
     @classmethod
-    def _create_attachment(cls, name):
+    def _create_attachment(cls, name, partner=None):
+        """
+        Create an attachment for the partner model.
+        If no partner is provided, the default partner is used.
+
+        :param str name: The name of the attachment.
+        :param odoo.model.res_partner partner: The partner to attach the document to.
+
+        :return: The created attachment.
+        :rtype: odoo.model.ir.attachment
+        """
+        if not partner:
+            partner = cls.partner
         return cls.create_attachment(
             name=name,
             res_model=cls.partner_model._name,
-            res_id=cls.partner.id,
+            res_id=partner.id,
         )
 
     @classmethod
-    def _get_partner_directory(cls):
+    def _get_partner_directory(cls, partner=False):
+        if not partner:
+            partner = cls.partner
         return cls.directory_model.search(
             [
                 ("storage_id", "=", cls.storage.id),
                 ("res_model", "=", cls.partner_model._name),
-                ("res_id", "=", cls.partner.id),
+                ("res_id", "=", partner.id),
             ]
         )
 

odoo/addons/dms/tests/data/mail01.eml

@@ -1,5 +1,5 @@
 Return-Path: <123@acm.org>
-To: Manuel Lemos <directory+test@linux.local>
+To: Manuel Lemos <directory+test@dmsTest.com>
 Subject: Testing Manuel Lemos' MIME E-mail composing and sending PHP class: HTML message
 From: mlemos <345@acm.org>
 Reply-To: mlemos <456@acm.org>
@@ -7,7 +7,7 @@ Sender: mlemos@acm.org
 X-Mailer: http://www.phpclasses.org/mimemessage $Revision: 1.63 $ (mail)
 MIME-Version: 1.0
 Content-Type: multipart/mixed; boundary="652b8c4dcb00cdcdda1e16af36781caf"
-Message-ID: <20050430192829.0489.mlemos@acm.org>
+Message-ID: <20050430192829.0489.mlemos@dmsTest.com>
 Date: Sat, 30 Apr 2005 19:28:29 -0300
 
 

odoo/addons/dms/tests/data/mail02.eml

@@ -1,11 +1,11 @@
 MIME-Version: 1.0
 Received: by 10.220.191.194 with HTTP; Wed, 11 May 2011 12:27:12 -0700 (PDT)
 Date: Wed, 11 May 2011 13:27:12 -0600
-Delivered-To: jncjkq@gmail.com
+Delivered-To: jncjkq@dmsTest.com
 Message-ID: <BANLkTi=JCQO1h3ET-pT_PLEHejhSSYxTZw@mail.jncjkq.com>
 Subject: Test
 From: Bill Jncjkq <jncjkq@gmail.com>
-To: directory+test@jncjkq.net
+To: directory+test@dmsTest.com
 Content-Type: multipart/mixed; boundary=bcaec54eecc63acce904a3050f79
 
 --bcaec54eecc63acce904a3050f79

odoo/addons/dms/tests/test_benchmark.py

@@ -1,11 +1,12 @@
 # Copyright 2017-2019 MuK IT GmbH.
 # Copyright 2020 Creu Blanca
+# Copyright 2024 Subteno - Timothée Vannier (https://www.subteno.com).
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
 
 import logging
+import os
 import unittest
 
-from odoo.modules.module import get_module_resource
 from odoo.tests import common, tagged
 from odoo.tools import convert_file
 from odoo.tools.misc import profile
@@ -26,23 +27,15 @@ class BenchmarkTestCase(common.TransactionCase):
 
     @classmethod
     def _clean_existing_records(cls):
-        cls.env["dms.category"].search([]).unlink()
-        cls.env["dms.directory"].search([]).unlink()
-        cls.env["dms.storage"].search([]).unlink()
-        cls.env["dms.tag"].search([]).unlink()
+        cls.env["dms.category"].search([]).sudo().unlink()
+        cls.env["dms.directory"].search([]).sudo().unlink()
+        cls.env["dms.storage"].search([]).sudo().unlink()
+        cls.env["dms.tag"].search([]).sudo().unlink()
 
     @classmethod
     def _load(cls, module, *args):
-        convert_file(
-            cls.cr,
-            "dms",
-            get_module_resource(module, *args),
-            {},
-            "init",
-            False,
-            "test",
-            cls.registry._assertion_report,
-        )
+        path = str(os.path.join(module, *args))
+        convert_file(cls.env, "dms", args[-1], {}, "init", False, "test", path)
 
     @classmethod
     def _setup_benchmark_data(cls):
@@ -57,7 +50,7 @@ class BenchmarkTestCase(common.TransactionCase):
         formt = "{:7}" + "| {:28}" * columns
 
         result = formt.format(*data[0]) + "\n"
-        result += ("-" * 7) + (("+" + ("-") * 29) * columns) + "\n"
+        result += ("-" * 7) + (("+" + "-" * 29) * columns) + "\n"
         for row in data[1:]:
             result += formt.format(*row) + "\n"
         return result
@@ -66,12 +59,13 @@ class BenchmarkTestCase(common.TransactionCase):
         tfunc = track_function(return_tracking=True)(func)
         benchmark = []
         for item in args_list:
-            self.registry.clear_caches()
+            self.registry.clear_all_caches()
             args = item[0] if len(item) > 0 else []
             kwargs = item[1] if len(item) > 1 else {}
             tracking = tuple(tfunc(*args, **kwargs)[1][1:])
             # pylint: disable=too-few-format-args
-            benchmark.append("%sq %.3fs %.3fs %.3fs" % tracking)
+            # pyupgrade: disable=UP030
+            benchmark.append("{}sq {:.3f}s {:.3f}s {:.3f}s".format(*tracking))
         return benchmark
 
     # ----------------------------------------------------------
@@ -93,7 +87,6 @@ class BenchmarkTestCase(common.TransactionCase):
             "is_lock_editor",
             "permission_write",
             "permission_unlink",
-            "__last_update",
         ]
 
     def test_file_search_benchmark(self):
@@ -177,7 +170,7 @@ class BenchmarkTestCase(common.TransactionCase):
         model = self.env["dms.file"].with_context(bin_size=True)
 
         def test_function(model, limit):
-            return model.search([], limit=limit).name_get()
+            return model.search([], limit=limit).mapped("display_name")
 
         model_super = model.sudo()
         model_admin = model.with_user(admin_uid)
@@ -211,7 +204,9 @@ class BenchmarkTestCase(common.TransactionCase):
             test_function, args_demo
         )
 
-        info_message = "\n\nSearching and 'name_get' function with bin_size = True | "
+        info_message = (
+            "\n\nSearching and 'display_name' function with bin_size = True | "
+        )
         info_message += "Benchmark with Limit 1 / 80 / 500 / None (1500)\n\n"
         info_message += self._benchmark_table(
             [

odoo/addons/dms/tests/test_directory.py

@@ -1,6 +1,7 @@
 # Copyright 2017-2019 MuK IT GmbH.
 # Copyright 2020 Creu Blanca
 # Copyright 2021-2022 Tecnativa - Víctor Martínez
+# Copyright 2024 Subteno - Timothée Vannier (https://www.subteno.com).
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
 
 import os
@@ -26,50 +27,85 @@ class DirectoryTestCaseBase(StorageDatabaseBaseCase):
     def test_create_directory(self):
         root_directory = self.create_directory(storage=self.storage)
         sub_directory = self.create_directory(directory=root_directory)
-        self.assertEqual(sub_directory.storage_id.id, self.storage.id)
-        self.assertEqual(root_directory.count_directories, 1)
+        self.assertEqual(
+            sub_directory.storage_id.id,
+            self.storage.id,
+            msg="The storage of the subdirectory should be the same as "
+            "the root directory",
+        )
+        self.assertEqual(
+            root_directory.count_directories,
+            1,
+            msg="The root directory should have one subdirectory",
+        )
 
     @users("dms-manager", "dms-user")
     def test_copy_root_directory(self):
         copy_root_directory = self.directory.copy()
         copy_root_directory.flush_recordset()
         self.assertEqual(
-            self.directory.storage_id.id, copy_root_directory.storage_id.id
+            self.directory.storage_id.id,
+            copy_root_directory.storage_id.id,
+            msg="The storage of the copied root directory should be the same as the "
+            "original root directory",
         )
         self.assertEqual(
             self.directory.count_directories,
             copy_root_directory.count_directories,
+            msg="The copied root directory should have the same number of "
+            "subdirectories as the original root directory",
+        )
+        self.assertEqual(
+            self.directory.count_files,
+            copy_root_directory.count_files,
+            msg="The copied root directory should have the same number of "
+            "files as the original root directory",
         )
-        self.assertEqual(self.directory.count_files, copy_root_directory.count_files)
 
     @users("dms-manager", "dms-user")
     def test_copy_sub_directory(self):
         copy_sub_directory = self.subdirectory.copy()
         self.assertEqual(
-            self.subdirectory.storage_id.id, copy_sub_directory.storage_id.id
+            self.subdirectory.storage_id.id,
+            copy_sub_directory.storage_id.id,
+            msg="The storage of the copied subdirectory should be the same as the "
+            "original subdirectory",
         )
         self.assertEqual(
             self.subdirectory.count_directories,
             copy_sub_directory.count_directories,
+            msg="The copied subdirectory should have the same number of subdirectories "
+            "as the original subdirectory",
+        )
+        self.assertEqual(
+            self.subdirectory.count_files,
+            copy_sub_directory.count_files,
+            msg="The copied subdirectory should have the same number of "
+            "files as the original subdirectory",
         )
-        self.assertEqual(self.subdirectory.count_files, copy_sub_directory.count_files)
 
     @users("dms-manager", "dms-user")
     def test_rename_directory(self):
         path_names = self.subdirectory.complete_name
         self.directory.write({"name": "New Test Name %s" % self.env.user.login})
-        self.assertNotEqual(path_names, self.subdirectory.complete_name)
+        self.assertNotEqual(
+            path_names,
+            self.subdirectory.complete_name,
+            msg="The path name of the subdirectory should have changed",
+        )
 
     @users("dms-manager", "dms-user")
     def test_move_directory(self):
-        with self.assertRaises(UserError):
+        with self.assertRaises(UserError, msg="The root directory should not be moved"):
             self.directory.write(
                 {
                     "is_root_directory": False,
                     "parent_id": self.subdirectory.id,
                 }
             )
-        with self.assertRaises(UserError):
+        with self.assertRaises(
+            UserError, msg="The subdirectory storage should not be changed"
+        ):
             self.subdirectory.write(
                 {
                     "is_root_directory": True,
@@ -92,7 +128,7 @@ class DirectoryTestCaseBase(StorageDatabaseBaseCase):
                 "parent_id": new_directory.id,
             }
         )
-        with self.assertRaises(UserError):
+        with self.assertRaises(UserError, msg="The root directory should not be moved"):
             self.directory.storage_id = self.new_storage.id
             self.subdirectory.write(
                 {
@@ -109,8 +145,12 @@ class DirectoryTestCaseBase(StorageDatabaseBaseCase):
         sub_directory = self.create_directory(directory=root_directory)
         sub_files = self.create_file(directory=sub_directory)
         root_directory.unlink()
-        self.assertFalse(sub_directory.exists())
-        self.assertFalse(sub_files.exists())
+        self.assertFalse(
+            sub_directory.exists(), msg="The subdirectory should not exist anymore"
+        )
+        self.assertFalse(
+            sub_files.exists(), msg="The subfiles should not exist anymore"
+        )
 
     @users("dms-manager", "dms-user")
     @mute_logger("odoo.models.unlink")
@@ -119,15 +159,27 @@ class DirectoryTestCaseBase(StorageDatabaseBaseCase):
         sub_directory = self.create_directory(directory=root_directory)
         sub_files = self.create_file(directory=sub_directory)
         sub_directory.unlink()
-        self.assertTrue(root_directory.exists())
-        self.assertFalse(sub_files.exists())
+        self.assertTrue(
+            root_directory.exists(), msg="The root directory should still exist"
+        )
+        self.assertFalse(
+            sub_files.exists(), msg="The subfiles should not exist anymore"
+        )
 
     @users("dms-manager", "dms-user")
     def test_storage(self):
         root_directory = self.create_directory(storage=self.storage)
         sub_directory = self.create_directory(directory=root_directory)
-        self.assertEqual(sub_directory.storage_id.id, self.storage.id)
-        with self.assertRaises(UserError):
+        self.assertEqual(
+            sub_directory.storage_id.id,
+            self.storage.id,
+            msg="The storage "
+            "of the subdirectory should be the same as the root "
+            "directory",
+        )
+        with self.assertRaises(
+            UserError, msg="The storage of the root directory should " "not be changed"
+        ):
             root_directory.write({"storage_id": self.new_storage.id})
 
     @users("dms-manager", "dms-user")
@@ -135,72 +187,111 @@ class DirectoryTestCaseBase(StorageDatabaseBaseCase):
         self.directory.toggle_starred()
         self.subdirectory.write({"starred": True})
         starred = self.directory_model.search([("starred", "=", True)])
-        self.assertIn(self.directory.id, starred.ids)
-        self.assertIn(self.subdirectory.id, starred.ids)
+        self.assertIn(
+            self.directory.id, starred.ids, msg="The directory should be starred"
+        )
+        self.assertIn(
+            self.subdirectory.id, starred.ids, msg="The subdirectory should be starred"
+        )
 
     @users("dms-manager", "dms-user")
     def test_count_directories(self):
-        self.assertTrue(self.directory.count_directories)
+        self.assertTrue(
+            self.directory.count_directories,
+            msg="The directory should have subdirectories",
+        )
 
     @users("dms-manager", "dms-user")
     def test_count_files(self):
-        self.assertTrue(self.subdirectory.count_files)
+        self.assertTrue(
+            self.subdirectory.count_files, msg="The subdirectory should have files"
+        )
 
     @users("dms-manager", "dms-user")
     def test_count_elements(self):
-        self.assertTrue(self.directory.count_elements)
+        self.assertTrue(
+            self.directory.count_elements,
+            msg="The directory should have subdirectories and files",
+        )
 
     @users("dms-manager", "dms-user")
     def test_count_total_directories(self):
-        self.assertTrue(self.directory.count_total_directories)
+        self.assertTrue(
+            self.directory.count_total_directories,
+            msg="The directory should have subdirectories",
+        )
 
     @users("dms-manager", "dms-user")
     def test_count_count_total_files(self):
-        self.assertTrue(self.directory.count_total_files)
+        self.assertTrue(
+            self.directory.count_total_files, msg="The directory should have files"
+        )
 
     @users("dms-manager", "dms-user")
     def test_count_total_elements(self):
-        self.assertTrue(self.directory.count_total_elements)
+        self.assertTrue(
+            self.directory.count_total_elements,
+            msg="The directory should have subdirectories and files",
+        )
 
     @users("dms-manager", "dms-user")
     def test_size(self):
-        self.assertTrue(self.directory.size)
+        self.assertTrue(self.directory.size, msg="The directory should have a size")
 
     @users("dms-manager", "dms-user")
     def test_name_get(self):
         directory = self.subdirectory.with_context(dms_directory_show_path=True)
         self.assertTrue(
-            "/" in directory.with_context(dms_directory_show_path=True).name_get()[0][1]
+            "/" in directory.with_context(dms_directory_show_path=True).display_name,
+            msg="The path should be displayed in the directory name",
         )
 
     @users("dms-manager", "dms-user")
     def test_name_search(self):
         directories = self.directory_model.name_search("/")
-        self.assertTrue(len(directories))
+        self.assertTrue(len(directories), msg="The root directory should be found")
 
     @users("dms-manager", "dms-user")
     def test_search_panel(self):
         self.assertTrue(
-            self.directory_model.search_panel_select_multi_range("parent_id")
+            self.directory_model.search_panel_select_multi_range("parent_id"),
+            msg="The parent_id field should be a multi range field",
         )
         self.assertTrue(
-            self.directory_model.search_panel_select_multi_range("category_id")
+            self.directory_model.search_panel_select_multi_range("category_id"),
+            msg="The category_id field should be a multi range field",
+        )
+        self.assertTrue(
+            self.directory_model.search_panel_select_multi_range("tag_ids"),
+            msg="The tag_ids field should be a multi range field",
         )
-        self.assertTrue(self.directory_model.search_panel_select_multi_range("tag_ids"))
 
 
 class DirectoryMailTestCase(StorageDatabaseBaseCase):
-    def setUp(self):
-        super().setUp()
-        self.params = self.env["ir.config_parameter"].sudo()
-        self.params.set_param("mail.catchall.domain", "mydomain.com")
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.params = cls.env["ir.config_parameter"].sudo()
+        cls.params.set_param("mail.catchall.domain", "dmstest.com")
+        domain = cls.env["mail.alias.domain"].create({"name": "dmstest.com"})
+        cls.env["mail.alias"].create(
+            {
+                "alias_model_id": cls.env["ir.model"]
+                .search([("model", "=", "dms.directory")])
+                .id,
+                "alias_domain_id": domain.id,
+            }
+        )
 
     @mute_logger("odoo.addons.mail.mail_thread")
     def test_mail_alias_files(self):
         self.directory.write({"alias_process": "files", "alias_name": "directory+test"})
-        with open(os.path.join(_path, "tests", "data", "mail01.eml"), "r") as file:
+        self._handle_mail_reception()
+
+    def _handle_mail_reception(self):
+        with open(os.path.join(_path, "tests", "data", "mail01.eml")) as file:
             self.env["mail.thread"].message_process(None, file.read())
-        with open(os.path.join(_path, "tests", "data", "mail02.eml"), "r") as file:
+        with open(os.path.join(_path, "tests", "data", "mail02.eml")) as file:
             self.env["mail.thread"].message_process(None, file.read())
 
     @mute_logger("odoo.addons.mail.mail_thread")
@@ -208,7 +299,4 @@ class DirectoryMailTestCase(StorageDatabaseBaseCase):
         self.directory.write(
             {"alias_process": "directory", "alias_name": "directory+test"}
         )
-        with open(os.path.join(_path, "tests", "data", "mail01.eml"), "r") as file:
-            self.env["mail.thread"].message_process(None, file.read())
-        with open(os.path.join(_path, "tests", "data", "mail02.eml"), "r") as file:
-            self.env["mail.thread"].message_process(None, file.read())
+        self._handle_mail_reception()