From 392ae4ac5236b506bf6b3e14678a5e930f29cdd8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rapha=C3=ABl=20Valyi?= <rvalyi@akretion.com>
Date: Fri, 16 Aug 2024 01:43:06 +0000
Subject: [PATCH] [REF] l10n_br_fiscal: extract l10n_br_fiscal_notification

---
 l10n_br_fiscal/__manifest__.py                |  2 -
 l10n_br_fiscal/hooks.py                       |  1 -
 l10n_br_fiscal/models/document.py             | 73 ----------------
 l10n_br_fiscal/views/document_view.xml        |  4 +-
 .../views/l10n_br_fiscal_action.xml           | 19 +----
 l10n_br_fiscal/views/l10n_br_fiscal_menu.xml  |  9 --
 l10n_br_fiscal_notification/__init__.py       |  1 +
 l10n_br_fiscal_notification/__manifest__.py   | 28 +++++++
 .../data/l10n_br_fiscal_email_template.xml    |  0
 .../demo/l10n_br_fiscal_document_email.xml    |  8 +-
 .../models/__init__.py                        |  1 +
 .../models/document.py                        | 83 +++++++++++++++++++
 .../views/document_email_view.xml             |  0
 .../views/l10n_br_fiscal_action.xml           | 21 +++++
 .../views/l10n_br_fiscal_menu.xml             | 13 +++
 .../odoo/addons/l10n_br_fiscal_notification   |  1 +
 setup/l10n_br_fiscal_notification/setup.py    |  6 ++
 17 files changed, 161 insertions(+), 109 deletions(-)
 create mode 100644 l10n_br_fiscal_notification/__init__.py
 create mode 100644 l10n_br_fiscal_notification/__manifest__.py
 rename {l10n_br_fiscal => l10n_br_fiscal_notification}/data/l10n_br_fiscal_email_template.xml (100%)
 rename {l10n_br_fiscal => l10n_br_fiscal_notification}/demo/l10n_br_fiscal_document_email.xml (81%)
 create mode 100644 l10n_br_fiscal_notification/models/__init__.py
 create mode 100644 l10n_br_fiscal_notification/models/document.py
 rename {l10n_br_fiscal => l10n_br_fiscal_notification}/views/document_email_view.xml (100%)
 create mode 100644 l10n_br_fiscal_notification/views/l10n_br_fiscal_action.xml
 create mode 100644 l10n_br_fiscal_notification/views/l10n_br_fiscal_menu.xml
 create mode 120000 setup/l10n_br_fiscal_notification/odoo/addons/l10n_br_fiscal_notification
 create mode 100644 setup/l10n_br_fiscal_notification/setup.py

diff --git a/l10n_br_fiscal/__manifest__.py b/l10n_br_fiscal/__manifest__.py
index 998e94370538..5c216a2071b3 100644
--- a/l10n_br_fiscal/__manifest__.py
+++ b/l10n_br_fiscal/__manifest__.py
@@ -21,7 +21,6 @@
         "security/ir.model.access.csv",
         # Data
         # Some data is being loaded via post_init_hook in hook file
-        "data/l10n_br_fiscal_email_template.xml",
         "data/l10n_br_fiscal_data.xml",
         "data/uom_data.xml",
         "data/uom_alternative_data.xml",
@@ -62,7 +61,6 @@
         "views/product_genre_view.xml",
         "views/document_type_view.xml",
         "views/document_serie_view.xml",
-        "views/document_email_view.xml",
         "views/simplified_tax_view.xml",
         "views/simplified_tax_range_view.xml",
         "views/operation_view.xml",
diff --git a/l10n_br_fiscal/hooks.py b/l10n_br_fiscal/hooks.py
index c1b0ae474d87..5ba390c0b053 100644
--- a/l10n_br_fiscal/hooks.py
+++ b/l10n_br_fiscal/hooks.py
@@ -55,7 +55,6 @@ def post_init_hook(cr, registry):
             "demo/fiscal_document_nfse_demo.xml",
             "demo/fiscal_operation_demo.xml",
             "demo/subsequent_operation_demo.xml",
-            "demo/l10n_br_fiscal_document_email.xml",
             "demo/res_users_demo.xml",
             "demo/icms_tax_definition_demo.xml",
         ]
diff --git a/l10n_br_fiscal/models/document.py b/l10n_br_fiscal/models/document.py
index 58e72ef08323..51935ba8f3c5 100644
--- a/l10n_br_fiscal/models/document.py
+++ b/l10n_br_fiscal/models/document.py
@@ -404,36 +404,6 @@ def action_create_return(self):
 
         return action
 
-    def _get_email_template(self, state):
-        self.ensure_one()
-        return self.document_type_id.document_email_ids.search(
-            [
-                "|",
-                ("state_edoc", "=", False),
-                ("state_edoc", "=", state),
-                ("issuer", "=", self.issuer),
-                "|",
-                ("document_type_id", "=", False),
-                ("document_type_id", "=", self.document_type_id.id),
-            ],
-            limit=1,
-            order="state_edoc, document_type_id",
-        ).mapped("email_template_id")
-
-    def send_email(self, state):
-        self.ensure_one()
-        email_template = self._get_email_template(state)
-        if email_template:
-            email_template.with_context(
-                default_attachment_ids=self._get_mail_attachment()
-            ).send_mail(self.id)
-
-    def _after_change_state(self, old_state, new_state):
-        self.ensure_one()
-        result = super()._after_change_state(old_state, new_state)
-        self.send_email(new_state)
-        return result
-
     @api.onchange("fiscal_operation_id")
     def _onchange_fiscal_operation_id(self):
         result = super()._onchange_fiscal_operation_id()
@@ -510,46 +480,3 @@ def cancel_edoc(self):
                 "associated documents have already been authorized."
             )
             raise UserWarning(message)
-
-    def _get_mail_attachment(self):
-        self.ensure_one()
-        attachment_ids = []
-        if self.state_edoc == SITUACAO_EDOC_AUTORIZADA:
-            if self.file_report_id:
-                attachment_ids.append(self.file_report_id.id)
-            if self.authorization_file_id:
-                attachment_ids.append(self.authorization_file_id.id)
-        return attachment_ids
-
-    def action_send_email(self):
-        """Open a window to compose an email, with the fiscal document_type
-        template message loaded by default
-        """
-        self.ensure_one()
-        template = self._get_email_template(self.state)
-        compose_form = self.env.ref("mail.email_compose_message_wizard_form", False)
-        lang = self.env.context.get("lang")
-        if template and template.lang:
-            lang = template._render_template(template.lang, self._name, [self.id])
-        self = self.with_context(lang=lang)
-        ctx = dict(
-            default_model="l10n_br_fiscal.document",
-            default_res_id=self.id,
-            default_use_template=bool(template),
-            default_attachment_ids=self._get_mail_attachment(),
-            default_template_id=template and template.id or False,
-            default_composition_mode="comment",
-            model_description=self.document_type_id.name or self._name,
-            force_email=True,
-        )
-        return {
-            "name": _("Send Fiscal Document Email Notification"),
-            "type": "ir.actions.act_window",
-            "view_type": "form",
-            "view_mode": "form",
-            "res_model": "mail.compose.message",
-            "views": [(compose_form.id, "form")],
-            "view_id": compose_form.id,
-            "target": "new",
-            "context": ctx,
-        }
diff --git a/l10n_br_fiscal/views/document_view.xml b/l10n_br_fiscal/views/document_view.xml
index b18079200a01..5ac1ffe7f087 100644
--- a/l10n_br_fiscal/views/document_view.xml
+++ b/l10n_br_fiscal/views/document_view.xml
@@ -171,13 +171,13 @@
                         groups="l10n_br_fiscal.group_user"
                         attrs="{'invisible': [('state_edoc', '!=', 'autorizada')]}"
                     />
-            <button
+            <!--button
                         name="action_send_email"
                         type="object"
                         string="Send Email"
                         groups="l10n_br_fiscal.group_user"
                         attrs="{'invisible': [('state_edoc', 'not in', ('autorizada', 'cancelada', 'denegada'))]}"
-                    />
+                    /-->
             <field
                         name="state_edoc"
                         widget="statusbar"
diff --git a/l10n_br_fiscal/views/l10n_br_fiscal_action.xml b/l10n_br_fiscal/views/l10n_br_fiscal_action.xml
index fb45ee42d652..e43ec7b221b0 100644
--- a/l10n_br_fiscal/views/l10n_br_fiscal_action.xml
+++ b/l10n_br_fiscal/views/l10n_br_fiscal_action.xml
@@ -372,24 +372,7 @@
         </field>
     </record>
 
-    <!-- Fiscal Document Email -->
-    <record id="document_email_action" model="ir.actions.act_window">
-        <field name="name">Fiscal Document Email</field>
-        <field name="type">ir.actions.act_window</field>
-        <field name="res_model">l10n_br_fiscal.document.email</field>
-        <field name="view_mode">tree,form</field>
-        <field name="view_id" ref="document_email_tree" />
-        <field name="help" type="html">
-            <p class="o_view_nocontent_smiling_face">
-              Add a new Fiscal Document Email Definition
-            </p><p>
-              All Document Email Definitions supported by the Brazilian
-              Fiscal module for Odoo.
-            </p>
-        </field>
-    </record>
-
-    <!-- Simplefied Tax -->
+    <!-- Simplified Tax -->
     <record id="simplified_tax_action" model="ir.actions.act_window">
         <field name="name">Simplified Tax</field>
         <field name="type">ir.actions.act_window</field>
diff --git a/l10n_br_fiscal/views/l10n_br_fiscal_menu.xml b/l10n_br_fiscal/views/l10n_br_fiscal_menu.xml
index e38a9f9e4b58..f3017a1429ba 100644
--- a/l10n_br_fiscal/views/l10n_br_fiscal_menu.xml
+++ b/l10n_br_fiscal/views/l10n_br_fiscal_menu.xml
@@ -570,15 +570,6 @@
         sequence="20"
     />
 
-    <!-- Fiscal Document Email -->
-    <menuitem
-        id="document_email_menu"
-        action="document_email_action"
-        groups="l10n_br_fiscal.group_manager"
-        parent="others_config_menu"
-        sequence="30"
-    />
-
     <!-- Comment -->
     <menuitem
         id="comment_menu"
diff --git a/l10n_br_fiscal_notification/__init__.py b/l10n_br_fiscal_notification/__init__.py
new file mode 100644
index 000000000000..0650744f6bc6
--- /dev/null
+++ b/l10n_br_fiscal_notification/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/l10n_br_fiscal_notification/__manifest__.py b/l10n_br_fiscal_notification/__manifest__.py
new file mode 100644
index 000000000000..d5f7dae1abe7
--- /dev/null
+++ b/l10n_br_fiscal_notification/__manifest__.py
@@ -0,0 +1,28 @@
+# License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
+
+{
+    "name": "Fiscal Document Notifications",
+    "summary": "Define fiscal document notifications",
+    "category": "Localisation",
+    "license": "AGPL-3",
+    "author": "KMEE, Odoo Community Association (OCA)",
+    "maintainers": ["mileo"],
+    "website": "https://github.com/OCA/l10n-brazil",
+    "development_status": "Production/Stable",
+    "version": "14.0.1.0.0",
+    "depends": [
+        "l10n_br_fiscal",
+    ],
+    "data": [
+        # Data
+        "data/l10n_br_fiscal_email_template.xml",
+        # Views
+        "views/document_email_view.xml",
+        # Actions
+        "views/l10n_br_fiscal_action.xml",
+        # Menus
+        "views/l10n_br_fiscal_menu.xml",
+    ],
+    "demo": ["demo/l10n_br_fiscal_document_email.xml"],
+    "installable": True,
+}
diff --git a/l10n_br_fiscal/data/l10n_br_fiscal_email_template.xml b/l10n_br_fiscal_notification/data/l10n_br_fiscal_email_template.xml
similarity index 100%
rename from l10n_br_fiscal/data/l10n_br_fiscal_email_template.xml
rename to l10n_br_fiscal_notification/data/l10n_br_fiscal_email_template.xml
diff --git a/l10n_br_fiscal/demo/l10n_br_fiscal_document_email.xml b/l10n_br_fiscal_notification/demo/l10n_br_fiscal_document_email.xml
similarity index 81%
rename from l10n_br_fiscal/demo/l10n_br_fiscal_document_email.xml
rename to l10n_br_fiscal_notification/demo/l10n_br_fiscal_document_email.xml
index 0e18a8450635..035ba6386247 100644
--- a/l10n_br_fiscal/demo/l10n_br_fiscal_document_email.xml
+++ b/l10n_br_fiscal_notification/demo/l10n_br_fiscal_document_email.xml
@@ -9,7 +9,7 @@
         <field name="issuer">company</field>
         <field
             name="email_template_id"
-            ref="l10n_br_fiscal.fiscal_document_change_state_template"
+            ref="l10n_br_fiscal_notification.fiscal_document_change_state_template"
         />
         <field name="company_id" ref="l10n_br_base.empresa_lucro_presumido" />
     </record>
@@ -22,7 +22,7 @@
         <field name="issuer">company</field>
         <field
             name="email_template_id"
-            ref="l10n_br_fiscal.fiscal_document_change_state_template_nfse"
+            ref="l10n_br_fiscal_notification.fiscal_document_change_state_template_nfse"
         />
         <field name="company_id" ref="l10n_br_base.empresa_lucro_presumido" />
     </record>
@@ -34,7 +34,7 @@
         <field name="issuer">company</field>
         <field
             name="email_template_id"
-            ref="l10n_br_fiscal.fiscal_document_change_state_template"
+            ref="l10n_br_fiscal_notification.fiscal_document_change_state_template"
         />
         <field name="company_id" ref="l10n_br_base.empresa_simples_nacional" />
     </record>
@@ -47,7 +47,7 @@
         <field name="issuer">company</field>
         <field
             name="email_template_id"
-            ref="l10n_br_fiscal.fiscal_document_change_state_template_nfse"
+            ref="l10n_br_fiscal_notification.fiscal_document_change_state_template_nfse"
         />
         <field name="company_id" ref="l10n_br_base.empresa_simples_nacional" />
     </record>
diff --git a/l10n_br_fiscal_notification/models/__init__.py b/l10n_br_fiscal_notification/models/__init__.py
new file mode 100644
index 000000000000..9c01ea2470e4
--- /dev/null
+++ b/l10n_br_fiscal_notification/models/__init__.py
@@ -0,0 +1 @@
+from . import document
diff --git a/l10n_br_fiscal_notification/models/document.py b/l10n_br_fiscal_notification/models/document.py
new file mode 100644
index 000000000000..9defb8d187ff
--- /dev/null
+++ b/l10n_br_fiscal_notification/models/document.py
@@ -0,0 +1,83 @@
+# Copyright (C) 2019  KMEE
+# License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
+
+from odoo import _, models
+
+from odoo.addons.l10n_br_fiscal.constants.fiscal import SITUACAO_EDOC_AUTORIZADA
+
+
+class Document(models.Model):
+    _inherit = "l10n_br_fiscal.document"
+
+    def _get_email_template(self, state):
+        self.ensure_one()
+        return self.document_type_id.document_email_ids.search(
+            [
+                "|",
+                ("state_edoc", "=", False),
+                ("state_edoc", "=", state),
+                ("issuer", "=", self.issuer),
+                "|",
+                ("document_type_id", "=", False),
+                ("document_type_id", "=", self.document_type_id.id),
+            ],
+            limit=1,
+            order="state_edoc, document_type_id",
+        ).mapped("email_template_id")
+
+    def send_email(self, state):
+        self.ensure_one()
+        email_template = self._get_email_template(state)
+        if email_template:
+            email_template.with_context(
+                default_attachment_ids=self._get_mail_attachment()
+            ).send_mail(self.id)
+
+    def _after_change_state(self, old_state, new_state):
+        self.ensure_one()
+        result = super()._after_change_state(old_state, new_state)
+        self.send_email(new_state)
+        return result
+
+    def _get_mail_attachment(self):
+        self.ensure_one()
+        attachment_ids = []
+        if self.state_edoc == SITUACAO_EDOC_AUTORIZADA:
+            if self.file_report_id:
+                attachment_ids.append(self.file_report_id.id)
+            if self.authorization_file_id:
+                attachment_ids.append(self.authorization_file_id.id)
+        return attachment_ids
+
+    def action_send_email(self):
+        """Open a window to compose an email, with the fiscal document_type
+        template message loaded by default
+        """
+        self.ensure_one()
+        template = self._get_email_template(self.state)
+        compose_form = self.env.ref("mail.email_compose_message_wizard_form", False)
+        lang = self.env.context.get("lang")
+        if template and template.lang:
+            lang = template._render_template(template.lang, self._name, [self.id])
+        self = self.with_context(lang=lang)
+        ctx = dict(
+            default_model="l10n_br_fiscal.document",
+            default_res_id=self.id,
+            default_use_template=bool(template),
+            default_attachment_ids=self._get_mail_attachment(),
+            default_template_id=template and template.id or False,
+            default_composition_mode="comment",
+            model_description=self.document_type_id.name or self._name,
+            force_email=True,
+        )
+        return {
+            "name": _("Send Fiscal Document Email Notification"),
+            "type": "ir.actions.act_window",
+            "view_type": "form",
+            "view_mode": "form",
+            "res_model": "mail.compose.message",
+            "views": [(compose_form.id, "form")],
+            "view_id": compose_form.id,
+            "target": "new",
+            "context": ctx,
+        }
diff --git a/l10n_br_fiscal/views/document_email_view.xml b/l10n_br_fiscal_notification/views/document_email_view.xml
similarity index 100%
rename from l10n_br_fiscal/views/document_email_view.xml
rename to l10n_br_fiscal_notification/views/document_email_view.xml
diff --git a/l10n_br_fiscal_notification/views/l10n_br_fiscal_action.xml b/l10n_br_fiscal_notification/views/l10n_br_fiscal_action.xml
new file mode 100644
index 000000000000..b460d7d2c644
--- /dev/null
+++ b/l10n_br_fiscal_notification/views/l10n_br_fiscal_action.xml
@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<odoo>
+
+    <!-- Fiscal Document Email -->
+    <record id="document_email_action" model="ir.actions.act_window">
+        <field name="name">Fiscal Document Email</field>
+        <field name="type">ir.actions.act_window</field>
+        <field name="res_model">l10n_br_fiscal.document.email</field>
+        <field name="view_mode">tree,form</field>
+        <field name="view_id" ref="document_email_tree" />
+        <field name="help" type="html">
+            <p class="o_view_nocontent_smiling_face">
+              Add a new Fiscal Document Email Definition
+            </p><p>
+              All Document Email Definitions supported by the Brazilian
+              Fiscal module for Odoo.
+            </p>
+        </field>
+    </record>
+
+</odoo>
diff --git a/l10n_br_fiscal_notification/views/l10n_br_fiscal_menu.xml b/l10n_br_fiscal_notification/views/l10n_br_fiscal_menu.xml
new file mode 100644
index 000000000000..551b7d5922a7
--- /dev/null
+++ b/l10n_br_fiscal_notification/views/l10n_br_fiscal_menu.xml
@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<odoo>
+
+    <!-- Fiscal Document Email -->
+    <menuitem
+        id="document_email_menu"
+        action="document_email_action"
+        groups="l10n_br_fiscal.group_manager"
+        parent="l10n_br_fiscal.others_config_menu"
+        sequence="30"
+    />
+
+</odoo>
diff --git a/setup/l10n_br_fiscal_notification/odoo/addons/l10n_br_fiscal_notification b/setup/l10n_br_fiscal_notification/odoo/addons/l10n_br_fiscal_notification
new file mode 120000
index 000000000000..9a4d642825ad
--- /dev/null
+++ b/setup/l10n_br_fiscal_notification/odoo/addons/l10n_br_fiscal_notification
@@ -0,0 +1 @@
+../../../../l10n_br_fiscal_notification
\ No newline at end of file
diff --git a/setup/l10n_br_fiscal_notification/setup.py b/setup/l10n_br_fiscal_notification/setup.py
new file mode 100644
index 000000000000..28c57bb64031
--- /dev/null
+++ b/setup/l10n_br_fiscal_notification/setup.py
@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)