diff --git a/mail_activity_unlink_log/README.rst b/mail_activity_unlink_log/README.rst new file mode 100644 index 000000000..df7a57a11 --- /dev/null +++ b/mail_activity_unlink_log/README.rst @@ -0,0 +1,83 @@ +.. image:: https://odoo-community.org/readme-banner-image + :target: https://odoo-community.org/get-involved?utm_source=readme + :alt: Odoo Community Association + +============================== +Mail Activities: log on unlink +============================== + +.. + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! This file is generated by oca-gen-addon-readme !! + !! changes will be overwritten. !! + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! source digest: sha256:9196aedd490fc01c791256582122032365ce7d26a11a5c0e13b2ae66777b06c3 + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + +.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png + :target: https://odoo-community.org/page/development-status + :alt: Beta +.. |badge2| image:: https://img.shields.io/badge/license-AGPL--3-blue.png + :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html + :alt: License: AGPL-3 +.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fmail-lightgray.png?logo=github + :target: https://github.com/OCA/mail/tree/19.0/mail_activity_unlink_log + :alt: OCA/mail +.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png + :target: https://translation.odoo-community.org/projects/mail-19-0/mail-19-0-mail_activity_unlink_log + :alt: Translate me on Weblate +.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png + :target: https://runboat.odoo-community.org/builds?repo=OCA/mail&target_branch=19.0 + :alt: Try me on Runboat + +|badge1| |badge2| |badge3| |badge4| |badge5| + +This addon enforces the creation of a message when an activity is +unlinked. This change tries to avoid a possible bad practice of deletion +of an activity without notifying the deletion. + +**Table of contents** + +.. contents:: + :local: + +Bug Tracker +=========== + +Bugs are tracked on `GitHub Issues `_. +In case of trouble, please check there if your issue has already been reported. +If you spotted it first, help us to smash it by providing a detailed and welcomed +`feedback `_. + +Do not contact contributors directly about support or help with technical issues. + +Credits +======= + +Authors +------- + +* CreuBlanca + +Contributors +------------ + +- Enric Tobella +- Sumit Basak (`@TheSumitBasak `__) + +Maintainers +----------- + +This module is maintained by the OCA. + +.. image:: https://odoo-community.org/logo.png + :alt: Odoo Community Association + :target: https://odoo-community.org + +OCA, or the Odoo Community Association, is a nonprofit organization whose +mission is to support the collaborative development of Odoo features and +promote its widespread use. + +This module is part of the `OCA/mail `_ project on GitHub. + +You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. diff --git a/mail_activity_unlink_log/__init__.py b/mail_activity_unlink_log/__init__.py new file mode 100644 index 000000000..0650744f6 --- /dev/null +++ b/mail_activity_unlink_log/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/mail_activity_unlink_log/__manifest__.py b/mail_activity_unlink_log/__manifest__.py new file mode 100644 index 000000000..eea04bba9 --- /dev/null +++ b/mail_activity_unlink_log/__manifest__.py @@ -0,0 +1,14 @@ +# Copyright 2023 CreuBlanca +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). + +{ + "name": "Mail Activities: log on unlink", + "summary": """ + Leave a message when an activity is unlinked""", + "version": "19.0.1.0.0", + "license": "AGPL-3", + "author": "CreuBlanca,Odoo Community Association (OCA)", + "website": "https://github.com/OCA/mail", + "depends": ["mail"], + "data": ["data/unlink_message.xml"], +} diff --git a/mail_activity_unlink_log/data/unlink_message.xml b/mail_activity_unlink_log/data/unlink_message.xml new file mode 100644 index 000000000..e19d94c3e --- /dev/null +++ b/mail_activity_unlink_log/data/unlink_message.xml @@ -0,0 +1,38 @@ + + + + + Activities unlink + + + + + + + + + diff --git a/mail_activity_unlink_log/i18n/es.po b/mail_activity_unlink_log/i18n/es.po new file mode 100644 index 000000000..a599493d3 --- /dev/null +++ b/mail_activity_unlink_log/i18n/es.po @@ -0,0 +1,56 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * mail_activity_unlink_log +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 14.0\n" +"Report-Msgid-Bugs-To: \n" +"PO-Revision-Date: 2024-01-28 11:33+0000\n" +"Last-Translator: Ivorra78 \n" +"Language-Team: none\n" +"Language: es\n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: \n" +"Plural-Forms: nplurals=2; plural=n != 1;\n" +"X-Generator: Weblate 4.17\n" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "(originally assigned to" +msgstr "(originalmente asignado a" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "Original note:" +msgstr "Nota original:" + +#. module: mail_activity_unlink_log +#: model:mail.message.subtype,name:mail_activity_unlink_log.mt_activities_unlink +msgid "Activities unlink" +msgstr "Desvincular actividades" + +#. module: mail_activity_unlink_log +#: model:ir.model,name:mail_activity_unlink_log.model_mail_activity +msgid "Activity" +msgstr "Actividades" + +#. module: mail_activity_unlink_log +#: model:ir.model,name:mail_activity_unlink_log.model_mail_activity_mixin +msgid "Activity Mixin" +msgstr "Mezclador de Actividad" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "deleted" +msgstr "borrado" + +#~ msgid "Display Name" +#~ msgstr "Mostrar Nombre" + +#~ msgid "ID" +#~ msgstr "ID" + +#~ msgid "Last Modified on" +#~ msgstr "Última Modificación el" diff --git a/mail_activity_unlink_log/i18n/it.po b/mail_activity_unlink_log/i18n/it.po new file mode 100644 index 000000000..d4744cce5 --- /dev/null +++ b/mail_activity_unlink_log/i18n/it.po @@ -0,0 +1,47 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * mail_activity_unlink_log +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 14.0\n" +"Report-Msgid-Bugs-To: \n" +"PO-Revision-Date: 2025-02-24 11:06+0000\n" +"Last-Translator: mymage \n" +"Language-Team: none\n" +"Language: it\n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: \n" +"Plural-Forms: nplurals=2; plural=n != 1;\n" +"X-Generator: Weblate 5.6.2\n" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "(originally assigned to" +msgstr "(assegnato originariamente a" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "Original note:" +msgstr "Nota origine:" + +#. module: mail_activity_unlink_log +#: model:mail.message.subtype,name:mail_activity_unlink_log.mt_activities_unlink +msgid "Activities unlink" +msgstr "Scollega attività" + +#. module: mail_activity_unlink_log +#: model:ir.model,name:mail_activity_unlink_log.model_mail_activity +msgid "Activity" +msgstr "Attività" + +#. module: mail_activity_unlink_log +#: model:ir.model,name:mail_activity_unlink_log.model_mail_activity_mixin +msgid "Activity Mixin" +msgstr "Mixin attività" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "deleted" +msgstr "cancellata" diff --git a/mail_activity_unlink_log/i18n/mail_activity_unlink_log.pot b/mail_activity_unlink_log/i18n/mail_activity_unlink_log.pot new file mode 100644 index 000000000..5644f1c31 --- /dev/null +++ b/mail_activity_unlink_log/i18n/mail_activity_unlink_log.pot @@ -0,0 +1,44 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * mail_activity_unlink_log +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 18.0\n" +"Report-Msgid-Bugs-To: \n" +"Last-Translator: \n" +"Language-Team: \n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: \n" +"Plural-Forms: \n" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "(originally assigned to" +msgstr "" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "Original note:" +msgstr "" + +#. module: mail_activity_unlink_log +#: model:mail.message.subtype,name:mail_activity_unlink_log.mt_activities_unlink +msgid "Activities unlink" +msgstr "" + +#. module: mail_activity_unlink_log +#: model:ir.model,name:mail_activity_unlink_log.model_mail_activity +msgid "Activity" +msgstr "" + +#. module: mail_activity_unlink_log +#: model:ir.model,name:mail_activity_unlink_log.model_mail_activity_mixin +msgid "Activity Mixin" +msgstr "" + +#. module: mail_activity_unlink_log +#: model_terms:ir.ui.view,arch_db:mail_activity_unlink_log.message_activity_unlink +msgid "deleted" +msgstr "" diff --git a/mail_activity_unlink_log/models/__init__.py b/mail_activity_unlink_log/models/__init__.py new file mode 100644 index 000000000..1ff388029 --- /dev/null +++ b/mail_activity_unlink_log/models/__init__.py @@ -0,0 +1 @@ +from . import mail_activity diff --git a/mail_activity_unlink_log/models/mail_activity.py b/mail_activity_unlink_log/models/mail_activity.py new file mode 100644 index 000000000..4a7410fa2 --- /dev/null +++ b/mail_activity_unlink_log/models/mail_activity.py @@ -0,0 +1,39 @@ +# Copyright 2023 CreuBlanca +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). + +from odoo import models + + +class MailActivity(models.Model): + _inherit = "mail.activity" + + def _action_done(self, *args, **kwargs): + return super( + MailActivity, self.with_context(activity_unlink_no_message=True) + )._action_done(*args, **kwargs) + + def unlink(self): + if not self.env.context.get("activity_unlink_no_message"): + for activity in self: + record = self.env[activity.res_model].browse(activity.res_id) + record.message_post_with_source( + "mail_activity_unlink_log.message_activity_unlink", + render_values={ + "activity": activity, + "display_assignee": activity.user_id != self.env.user, + }, + subtype_id=self.env["ir.model.data"]._xmlid_to_res_id( + "mail_activity_unlink_log.mt_activities_unlink" + ), + mail_activity_type_id=activity.activity_type_id.id, + ) + return super().unlink() + + +class MailActivityMixin(models.AbstractModel): + _inherit = "mail.activity.mixin" + + def unlink(self): + return super( + MailActivityMixin, self.with_context(activity_unlink_no_message=True) + ).unlink() diff --git a/mail_activity_unlink_log/pyproject.toml b/mail_activity_unlink_log/pyproject.toml new file mode 100644 index 000000000..4231d0ccc --- /dev/null +++ b/mail_activity_unlink_log/pyproject.toml @@ -0,0 +1,3 @@ +[build-system] +requires = ["whool"] +build-backend = "whool.buildapi" diff --git a/mail_activity_unlink_log/readme/CONTRIBUTORS.md b/mail_activity_unlink_log/readme/CONTRIBUTORS.md new file mode 100644 index 000000000..0d6def2ce --- /dev/null +++ b/mail_activity_unlink_log/readme/CONTRIBUTORS.md @@ -0,0 +1,2 @@ +- Enric Tobella +- Sumit Basak ([@TheSumitBasak](https://github.com/TheSumitBasak)) \ No newline at end of file diff --git a/mail_activity_unlink_log/readme/DESCRIPTION.md b/mail_activity_unlink_log/readme/DESCRIPTION.md new file mode 100644 index 000000000..6991c0122 --- /dev/null +++ b/mail_activity_unlink_log/readme/DESCRIPTION.md @@ -0,0 +1,3 @@ +This addon enforces the creation of a message when an activity is +unlinked. This change tries to avoid a possible bad practice of deletion +of an activity without notifying the deletion. diff --git a/mail_activity_unlink_log/static/description/icon.png b/mail_activity_unlink_log/static/description/icon.png new file mode 100644 index 000000000..3a0328b51 Binary files /dev/null and b/mail_activity_unlink_log/static/description/icon.png differ diff --git a/mail_activity_unlink_log/static/description/index.html b/mail_activity_unlink_log/static/description/index.html new file mode 100644 index 000000000..297ab3153 --- /dev/null +++ b/mail_activity_unlink_log/static/description/index.html @@ -0,0 +1,432 @@ + + + + + +README.rst + + + +
+ + + +Odoo Community Association + + +
+ + diff --git a/mail_activity_unlink_log/tests/__init__.py b/mail_activity_unlink_log/tests/__init__.py new file mode 100644 index 000000000..9ea08bfbb --- /dev/null +++ b/mail_activity_unlink_log/tests/__init__.py @@ -0,0 +1 @@ +from . import test_activity_unlink diff --git a/mail_activity_unlink_log/tests/test_activity_unlink.py b/mail_activity_unlink_log/tests/test_activity_unlink.py new file mode 100644 index 000000000..6e7bf27bd --- /dev/null +++ b/mail_activity_unlink_log/tests/test_activity_unlink.py @@ -0,0 +1,70 @@ +# Copyright 2023 CreuBlanca +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). + +from odoo.addons.base.tests.common import BaseCommon + + +class TestActivityUnlink(BaseCommon): + @classmethod + def setUpClass(cls): + super().setUpClass() + cls.unlink_subtype = cls.env.ref( + "mail_activity_unlink_log.mt_activities_unlink" + ) + + def test_done(self): + self.assertFalse( + self.partner.message_ids.filtered( + lambda r: r.subtype_id == self.unlink_subtype + ) + ) + self.activity = self.partner.activity_schedule( + act_type_xmlid="mail.mail_activity_data_todo" + ) + self.assertTrue(self.partner.activity_ids) + self.activity.action_done() + self.assertFalse(self.partner.activity_ids) + self.assertFalse( + self.partner.message_ids.filtered( + lambda r: r.subtype_id == self.unlink_subtype + ) + ) + + def test_unlink(self): + self.assertFalse( + self.partner.message_ids.filtered( + lambda r: r.subtype_id == self.unlink_subtype + ) + ) + self.activity = self.partner.activity_schedule( + act_type_xmlid="mail.mail_activity_data_todo" + ) + self.assertTrue(self.partner.activity_ids) + self.activity.unlink() + self.assertFalse(self.partner.activity_ids) + self.assertTrue( + self.partner.message_ids.filtered( + lambda r: r.subtype_id == self.unlink_subtype + ) + ) + + def test_mixin_unlink(self): + # No unlink messages initially + self.assertFalse( + self.partner.message_ids.filtered( + lambda r: r.subtype_id == self.unlink_subtype + ) + ) + + # Create an activity on the partner + self.partner.activity_schedule(act_type_xmlid="mail.mail_activity_data_todo") + self.assertTrue(self.partner.activity_ids) + + # Unlink the partner → triggers mail.activity.mixin.unlink + self.partner.unlink() + + # No unlink message should be posted because of the context flag + messages = self.env["mail.message"].search( + [("subtype_id", "=", self.unlink_subtype.id)] + ) + self.assertFalse(messages)