add account dms field

account_dms_field/__init__.py

@@ -0,0 +1 @@
+from . import models

account_dms_field/__manifest__.py

@@ -0,0 +1,12 @@
+{
+    "name": "Add dms field for account",
+    "version": "16.0.1.0.0",
+    "category": "Accounting/Accounting",
+    "website": "https://www.agenterp.com",
+    "author": "Agent ERP GmbH",
+    "depends": ["account", "dms_field"],
+    "data": ["views/account_move_view.xml"],
+    "demo": ["demo/account_dms_data.xml"],
+    "installable": True,
+    "license": "LGPL-3",
+}

account_dms_field/demo/account_dms_data.xml

@@ -0,0 +1,25 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<odoo>
+    <!-- Created a DMS access group. #Issue-249 -->
+    <record id="read_access_account_move_group" model="dms.access.group">
+        <field name="name">Everyone for Account DMS</field>
+        <field name="group_ids" eval="[(4, ref('account.group_account_invoice'))]" />
+        <field name="perm_create" eval="True" />
+        <field name="perm_write" eval="True" />
+        <field name="perm_unlink" eval="True" />
+    </record>
+
+    <!-- Created a DMS field template for account. #Issue-249 -->
+    <record id="field_template_account" model="dms.field.template">
+        <field name="name">Account</field>
+        <field name="storage_id" ref="dms.storage_demo" />
+        <field name="model_id" ref="account.model_account_move" />
+        <field name="user_field_id" ref="account.field_account_move__user_id" />
+        <field name="group_ids" eval="[(4, ref('read_access_account_move_group'))]" />
+    </record>
+
+    <!-- Created a account DMS directory. #Issue-249 -->
+    <function model="dms.field.template" name="_create_dms_directory">
+        <value eval="[ref('field_template_account')]" />
+    </function>
+</odoo>

account_dms_field/models/__init__.py

@@ -0,0 +1,3 @@
+from . import dms_field_template
+from . import account_move
+from . import dms_directory

account_dms_field/models/account_move.py

@@ -0,0 +1,6 @@
+from odoo import models
+
+
+class AccountMove(models.Model):
+    _name = "account.move"
+    _inherit = ["account.move", "dms.field.mixin"]

account_dms_field/models/dms_directory.py

@@ -0,0 +1,37 @@
+from odoo import _, api, models
+from odoo.exceptions import ValidationError
+
+from odoo.addons.dms.tools.file import check_name
+
+
+class DmsDirectory(models.Model):
+    _inherit = "dms.directory"
+
+    @api.constrains("name")
+    def _check_name(self):
+        """Override a method to update the record name for the account.move model.
+        #Issue-249"""
+        for record in self:
+            # BAD customization start. #Issue-249
+            name = record.name
+            # Updated the name of account move record due to an issue caused by a (/)
+            # in the record name when opening the temporary directory created in the
+            # check_name method. #Issue-249
+            if record.res_model == "account.move":
+                name = name.replace("/", "-")
+            # BAD customization end. #Issue-249
+            if self.env.context.get("check_name", True) and not check_name(name):
+                raise ValidationError(_("The directory name is invalid."))
+            if record.is_root_directory:
+                childs = record.sudo().storage_id.root_directory_ids.name_get()
+            else:
+                childs = record.sudo().parent_id.child_directory_ids.name_get()
+            if list(
+                filter(
+                    lambda child: child[1] == record.name and child[0] != record.id,
+                    childs,
+                )
+            ):
+                raise ValidationError(
+                    _("A directory with the same name already exists.")
+                )

account_dms_field/models/dms_field_template.py

@@ -0,0 +1,14 @@
+from odoo import models
+
+
+class DmsFieldTemplate(models.Model):
+    _inherit = "dms.field.template"
+
+    def _create_dms_directory(self):
+        """New method to define the directory automatically in demo data avoiding
+        error if it already exists."""
+        self.ensure_one()
+        if not self.dms_directory_ids:
+            self.with_context(
+                res_model=self._name, res_id=self.id
+            ).create_dms_directory()

account_dms_field/tests/__init__.py

@@ -0,0 +1 @@
+from . import test_account_dms_field

account_dms_field/tests/test_account_dms_field.py

@@ -0,0 +1,59 @@
+from odoo.addons.base.tests.common import TransactionCase
+
+
+class TestAccountDmsField(TransactionCase):
+    def setUp(self):
+        """Inherited method to create records. #Issue-249"""
+        super(TestAccountDmsField, self).setUp()
+        self.template = self.env.ref("account_dms_field.field_template_account")
+        self.storage = self.template.storage_id
+        self.access_group = self.template.group_ids
+        self.account_model = self.env["account.move"]
+        self.partner = self.env.ref("base.res_partner_12")
+        self.test_directory = self.env["dms.directory"].create(
+            {
+                "name": "Test Directory",
+                "parent_id": self.template.dms_directory_ids[0].id,
+                "storage_id": self.template.storage_id.id,
+            }
+        )
+
+    def test_01_account_document_directory(self):
+        """New method to test account order documents field value. #Issue-249"""
+        template_ctx = self.template.with_context(
+            res_model=self.template._name, res_id=self.template.id
+        )
+        template_ctx._create_dms_directory()
+        account_move = self.account_model.create(
+            {
+                "partner_id": self.partner.id,
+            }
+        )
+        account_move.invalidate_model()
+        directory = account_move.dms_directory_ids
+        # Assert that only one directory is created for the account move. #Issue-249
+        self.assertEqual(len(directory), 1, "Directory length must be 1.")
+        # Assert that the storage associated with the directory is the same as the
+        # template's storage. #Issue-249
+        self.assertEqual(
+            directory.storage_id,
+            self.storage,
+            "Account move directory storage is different from the template storage.",
+        )
+        # Assert that the custom access group is present in the directory's group
+        # list. #Issue-249
+        self.assertIn(
+            self.access_group,
+            directory.group_ids,
+            "Account move directory groups are different from the template groups.",
+        )
+        # Map the names of child directories related to the account move directory.
+        # Issue-249
+        child_directory_names = directory.mapped("child_directory_ids.name")
+        # Assert that a specific child directory, "Test Directory", exists. #Issue-249
+        self.assertIn(
+            "Test Directory",
+            child_directory_names,
+            "Test Directory is not in the child directory of the account move "
+            "directory.",
+        )

account_dms_field/views/account_move_view.xml

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<odoo>
+    <!-- Inherited form view to add new page. #Issue-249 -->
+    <record id="view_move_form_inherit_account_dms_field" model="ir.ui.view">
+        <field name="name">view.move.form.inherit.account.dms.field</field>
+        <field name="model">account.move</field>
+        <field name="inherit_id" ref="account.view_move_form" />
+        <field name="arch" type="xml">
+            <!-- Added a new document page. #Issue-249 -->
+            <xpath expr="//notebook" position="inside">
+                <page name="documents" string="Documents" attrs="{'invisible': [('id', '=', False)]}">
+                    <field name="dms_directory_ids" mode="dms_list" />
+                </page>
+            </xpath>
+        </field>
+    </record>
+</odoo>