Fix allow_mail_archived_partner causing system notifications to send emails
This commit is contained in:
@@ -7,20 +7,16 @@ from odoo import models, api
|
|||||||
class ResPartner(models.Model):
|
class ResPartner(models.Model):
|
||||||
_inherit = "res.partner"
|
_inherit = "res.partner"
|
||||||
|
|
||||||
_logger.info("=== RES.PARTNER CLASS LOADED (allow_mail_archived_partner) ===")
|
|
||||||
|
|
||||||
@api.model
|
@api.model
|
||||||
def _search(self, args, offset=0, limit=None, order=None, count=False, access_rights_uid=None):
|
def _search(self, args, offset=0, limit=None, order=None, count=False, access_rights_uid=None):
|
||||||
"""
|
"""
|
||||||
Allow archived partners when context permits.
|
Allow archived partners when context permits.
|
||||||
"""
|
"""
|
||||||
_logger.info("=== ResPartner._search ===")
|
_logger.debug("=== RES.PARTNER _search ===")
|
||||||
_logger.info(f"Search args: {args}")
|
_logger.debug(f"Search args before: {args}")
|
||||||
_logger.info(f"Context include_archived_partners: {self.env.context.get('include_archived_partners')}")
|
_logger.debug(f"Context include_archived: {self.env.context.get('include_archived_partners')}")
|
||||||
_logger.info(f"Context mail_notify_force: {self.env.context.get('mail_notify_force')}")
|
_logger.debug(f"Context mail_notify_force: {self.env.context.get('mail_notify_force')}")
|
||||||
_logger.info(f"Context force_email: {self.env.context.get('force_email')}")
|
_logger.debug(f"Context force_email: {self.env.context.get('force_email')}")
|
||||||
_logger.info(f"Context mark_invoice_as_sent: {self.env.context.get('mark_invoice_as_sent')}")
|
|
||||||
_logger.info(f"Full context active_test: {self.env.context.get('active_test')}")
|
|
||||||
|
|
||||||
# Check if we should include archived partners
|
# Check if we should include archived partners
|
||||||
include_archived = (
|
include_archived = (
|
||||||
@@ -31,7 +27,7 @@ class ResPartner(models.Model):
|
|||||||
)
|
)
|
||||||
|
|
||||||
if include_archived:
|
if include_archived:
|
||||||
_logger.info("✓ CONTEXT FLAGS FOUND - Allowing archived partners")
|
_logger.debug("Context flags found - allowing archived partners")
|
||||||
|
|
||||||
# Remove both active and partner_share filters
|
# Remove both active and partner_share filters
|
||||||
filtered_args = []
|
filtered_args = []
|
||||||
@@ -39,25 +35,25 @@ class ResPartner(models.Model):
|
|||||||
if isinstance(arg, (list, tuple)) and len(arg) == 3:
|
if isinstance(arg, (list, tuple)) and len(arg) == 3:
|
||||||
# Skip active filters
|
# Skip active filters
|
||||||
if arg[0] == "active":
|
if arg[0] == "active":
|
||||||
_logger.info(f" Removing active filter: {arg}")
|
_logger.debug(f"Removing active filter: {arg}")
|
||||||
continue
|
continue
|
||||||
# Skip partner_share filter for manual sends
|
# Skip partner_share filter for manual sends to archived partners
|
||||||
if arg[0] == "partner_share" and arg[1] == "=" and arg[2] is True:
|
if arg[0] == "partner_share" and arg[1] == "=" and arg[2] is True:
|
||||||
_logger.info(f" Removing partner_share filter: {arg}")
|
_logger.debug(f"Removing partner_share filter: {arg}")
|
||||||
continue
|
continue
|
||||||
|
|
||||||
filtered_args.append(arg)
|
filtered_args.append(arg)
|
||||||
|
|
||||||
args = filtered_args
|
args = filtered_args
|
||||||
self = self.with_context(active_test=False)
|
self = self.with_context(active_test=False)
|
||||||
_logger.info(f"Search args after cleanup: {args}")
|
_logger.debug(f"Search args after: {args}")
|
||||||
|
|
||||||
result = super()._search(args, offset, limit, order, count, access_rights_uid)
|
result = super()._search(args, offset, limit, order, count, access_rights_uid)
|
||||||
|
|
||||||
if not count:
|
if not count:
|
||||||
result_ids = list(result)
|
result_ids = list(result)
|
||||||
_logger.info(f"Search returned {len(result_ids)} results")
|
_logger.debug(f"Search returned {len(result_ids)} results")
|
||||||
if result_ids:
|
if result_ids:
|
||||||
_logger.info(f"First 5 result IDs: {result_ids[:5]}")
|
_logger.debug(f"First result IDs: {result_ids[:5]}")
|
||||||
|
|
||||||
return result
|
return result
|
||||||
Reference in New Issue
Block a user