diff --git a/mass_mailing_list_dynamic/__manifest__.py b/mass_mailing_list_dynamic/__manifest__.py
index 803719775..6e0e2439b 100644
--- a/mass_mailing_list_dynamic/__manifest__.py
+++ b/mass_mailing_list_dynamic/__manifest__.py
@@ -5,7 +5,7 @@
{
"name": "Dynamic Mass Mailing Lists",
"summary": "Mass mailing lists that get autopopulated",
- "version": "10.0.1.1.1",
+ "version": "10.0.1.2.0",
"category": "Marketing",
"website": "https://github.com/OCA/social",
"author": "Tecnativa, Odoo Community Association (OCA)",
diff --git a/mass_mailing_list_dynamic/models/mail_mass_mailing_list.py b/mass_mailing_list_dynamic/models/mail_mass_mailing_list.py
index 93a4fe673..47be1b65f 100644
--- a/mass_mailing_list_dynamic/models/mail_mass_mailing_list.py
+++ b/mass_mailing_list_dynamic/models/mail_mass_mailing_list.py
@@ -2,7 +2,7 @@
# Copyright 2017 Tecnativa - Jairo Llopis
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
-from odoo import fields, models
+from odoo import api, fields, models
from odoo.tools import safe_eval
@@ -29,6 +29,10 @@ class MassMailingList(models.Model):
required=True,
help="Filter partners to sync in this list",
)
+ is_synced = fields.Boolean(
+ help="Helper field to make the user aware of unsynced changes",
+ default=True,
+ )
def action_sync(self):
"""Sync contacts in dynamic lists."""
@@ -55,5 +59,11 @@ class MassMailingList(models.Model):
"list_id": one.id,
"partner_id": partner.id,
})
+ one.is_synced = True
# Invalidate cached contact count
self.invalidate_cache(["contact_nbr"], dynamic.ids)
+
+ @api.onchange("dynamic", "sync_method", "sync_domain")
+ def _onchange_dynamic(self):
+ if self.dynamic:
+ self.is_synced = False
diff --git a/mass_mailing_list_dynamic/tests/test_dynamic_lists.py b/mass_mailing_list_dynamic/tests/test_dynamic_lists.py
index ded412a68..eb781a59b 100644
--- a/mass_mailing_list_dynamic/tests/test_dynamic_lists.py
+++ b/mass_mailing_list_dynamic/tests/test_dynamic_lists.py
@@ -126,3 +126,14 @@ class DynamicListCase(common.SavepointCase):
self.partners[:1].write({
'email': 'test_mass_mailing_list_dynamic@example.org',
})
+
+ def test_is_synced(self):
+ self.list.dynamic = False
+ self.list._onchange_dynamic()
+ # It shouldn't change when list is reversed to normal
+ self.assertTrue(self.list.is_synced)
+ self.list.dynamic = True
+ self.list._onchange_dynamic()
+ self.assertFalse(self.list.is_synced)
+ self.list.action_sync()
+ self.assertTrue(self.list.is_synced)
diff --git a/mass_mailing_list_dynamic/views/mail_mass_mailing_list_view.xml b/mass_mailing_list_dynamic/views/mail_mass_mailing_list_view.xml
index 10046ef40..79dc523f9 100644
--- a/mass_mailing_list_dynamic/views/mail_mass_mailing_list_view.xml
+++ b/mass_mailing_list_dynamic/views/mail_mass_mailing_list_view.xml
@@ -15,6 +15,7 @@
+
@@ -22,11 +23,20 @@
+
@@ -61,6 +71,16 @@
/>
+
+
+ {'invisible': [('is_synced', '=', False)]}
+
+
+
+
+ ???
+
+
diff --git a/mass_mailing_list_dynamic/wizards/mail_mass_mailing_load_filter.py b/mass_mailing_list_dynamic/wizards/mail_mass_mailing_load_filter.py
index 9db897129..89834a2f0 100644
--- a/mass_mailing_list_dynamic/wizards/mail_mass_mailing_load_filter.py
+++ b/mass_mailing_list_dynamic/wizards/mail_mass_mailing_load_filter.py
@@ -12,7 +12,9 @@ class MassMailingLoadFilter(models.TransientModel):
comodel_name='ir.filters',
string="Filter to load",
required=True,
- domain=[('model_id', '=', 'res.partner')],
+ domain="[('model_id', '=', 'res.partner'), '|', "
+ "('user_id', '=', uid), ('user_id','=',False)]",
+ ondelete='cascade',
)
def load_filter(self):