Merge PR #4250 into 16.0

Signed-off-by SirAionTech
OCA · Jul 18, 2024 · 88374b5 · 88374b5
2 parents 77b0ae5 + cbf7cf1
commit 88374b5
Show file tree

Hide file tree

Showing 8 changed files with 46 additions and 2 deletions.
diff --git a/l10n_it_fatturapa/README.rst b/l10n_it_fatturapa/README.rst
@@ -170,6 +170,10 @@ Contributors
 
    -  Simone Rubino <[email protected]>
 
+-  `Stesi Consulting Srl <https://www.stesi.consulting/>`__:
+
+   -  Michele Di Croce <[email protected]>
+
 Maintainers
 -----------
 

diff --git a/l10n_it_fatturapa/__manifest__.py b/l10n_it_fatturapa/__manifest__.py
@@ -6,7 +6,7 @@
 
 {
     "name": "ITA - Fattura elettronica - Base",
-    "version": "16.0.1.1.0",
+    "version": "16.0.1.1.1",
     "category": "Localization/Italy",
     "summary": "Fatture elettroniche",
     "author": "Davide Corio, Agile Business Group, Innoviu, "

diff --git a/l10n_it_fatturapa/migrations/16.0.1.1.1/post-migrate.py b/l10n_it_fatturapa/migrations/16.0.1.1.1/post-migrate.py
@@ -0,0 +1,10 @@
+#  Copyright 2024 Michele Di Croce - Stesi Consulting srl
+#  License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from openupgradelib import openupgrade
+
+
+@openupgrade.migrate()
+def migrate(env, version):
+    fatturapa_attachments = env["fatturapa.attachments"].search([])
+    fatturapa_attachments._l10n_it_link_attachments()
diff --git a/l10n_it_fatturapa/models/__init__.py b/l10n_it_fatturapa/models/__init__.py
@@ -1,7 +1,7 @@
 # Copyright 2014 Davide Corio <[email protected]>
 
+from . import link_e_invoice_ir_attachment
 from . import account
 from . import company
 from . import partner
 from . import ir_attachment
-from . import link_e_invoice_ir_attachment
diff --git a/l10n_it_fatturapa/models/account.py b/l10n_it_fatturapa/models/account.py
@@ -217,6 +217,7 @@ class FatturaAttachments(models.Model):
     _name = "fatturapa.attachments"
     _description = "E-invoice attachments"
     _inherits = {"ir.attachment": "ir_attachment_id"}
+    _inherit = ["l10n_it_fatturapa.attachment.e_invoice.link"]
 
     ir_attachment_id = fields.Many2one(
         "ir.attachment", "Attachment", required=True, ondelete="cascade"

diff --git a/l10n_it_fatturapa/readme/CONTRIBUTORS.md b/l10n_it_fatturapa/readme/CONTRIBUTORS.md
@@ -19,3 +19,5 @@
   > - Giovanni Serra \<<[email protected]>\>
 - [Aion Tech](https://aiontech.company/):
   - Simone Rubino \<<[email protected]>\>
+- [Stesi Consulting](https://www.stesi.consulting):
+  - Michele Di Croce \<<[email protected]>\>
diff --git a/l10n_it_fatturapa/static/description/index.html b/l10n_it_fatturapa/static/description/index.html
@@ -509,6 +509,11 @@ <h2><a class="toc-backref" href="#toc-entry-7">Contributors</a></h2>
 <li>Simone Rubino &lt;<a class="reference external" href="mailto:simone.rubino&#64;aion-tech.it">simone.rubino&#64;aion-tech.it</a>&gt;</li>
 </ul>
 </li>
+<li><p class="first"><a class="reference external" href="https://www.stesi.consulting">Stesi Consulting</a>:</p>
+<ul class="simple">
+<li>Michele Di Croce &lt;<a class="reference external" href="mailto:dicroce.m&#64;stesi.consulting">dicroce.m&#64;stesi.consulting</a>&gt;</li>
+</ul>
+</li>
 </ul>
 </div>
 <div class="section" id="maintainers">

diff --git a/l10n_it_fatturapa_in/tests/test_import_fatturapa_xml.py b/l10n_it_fatturapa_in/tests/test_import_fatturapa_xml.py
@@ -1134,6 +1134,28 @@ def test_access_other_user_e_invoice(self):
         e_invoice = invoices.fatturapa_attachment_in_id
         self.assertTrue(e_invoice.ir_attachment_id.read())
 
+    def test_access_other_user_e_invoice_attachments(self):
+        """A user can see the e-invoice attachments created by other users."""
+        # Arrange
+        access_right_group_xmlid = "base.group_erp_manager"
+        user = self.env.user
+        user.groups_id -= self.env.ref("base.group_system")
+        user.groups_id -= self.env.ref(access_right_group_xmlid)
+        other_user = user.copy(default={"login": "attachment_user"})
+        # pre-condition
+        self.assertFalse(user.has_group(access_right_group_xmlid))
+        self.assertNotEqual(user, other_user)
+        import_action = self.run_wizard(
+            "access_other_user_e_invoice_attachments", "IT02780790107_11004.xml"
+        )
+        # Assert
+        with self.with_user(other_user.login):
+            invoices = self.env[import_action["res_model"]].search(
+                import_action["domain"]
+            )
+            e_invoice = invoices.fatturapa_doc_attachments
+            self.assertTrue(e_invoice.ir_attachment_id.read())
+
 
 class TestFatturaPAEnasarco(FatturapaCommon):
     def setUp(self):