From cb52e822d7d3157cf673d711db8e43c95492eb35 Mon Sep 17 00:00:00 2001 From: Kristof Bernaert Date: Mon, 26 Jan 2026 02:47:38 +0100 Subject: [PATCH] Fix allow_mail_archived_partner causing system notifications to send emails --- .../models/res_partner.py | 28 ++++++++----------- 1 file changed, 12 insertions(+), 16 deletions(-) diff --git a/allow_mail_archived_partner/models/res_partner.py b/allow_mail_archived_partner/models/res_partner.py index 39de529..2c264a6 100755 --- a/allow_mail_archived_partner/models/res_partner.py +++ b/allow_mail_archived_partner/models/res_partner.py @@ -7,20 +7,16 @@ from odoo import models, api class ResPartner(models.Model): _inherit = "res.partner" - _logger.info("=== RES.PARTNER CLASS LOADED (allow_mail_archived_partner) ===") - @api.model def _search(self, args, offset=0, limit=None, order=None, count=False, access_rights_uid=None): """ Allow archived partners when context permits. """ - _logger.info("=== ResPartner._search ===") - _logger.info(f"Search args: {args}") - _logger.info(f"Context include_archived_partners: {self.env.context.get('include_archived_partners')}") - _logger.info(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.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')}") + _logger.debug("=== RES.PARTNER _search ===") + _logger.debug(f"Search args before: {args}") + _logger.debug(f"Context include_archived: {self.env.context.get('include_archived_partners')}") + _logger.debug(f"Context mail_notify_force: {self.env.context.get('mail_notify_force')}") + _logger.debug(f"Context force_email: {self.env.context.get('force_email')}") # Check if we should include archived partners include_archived = ( @@ -31,7 +27,7 @@ class ResPartner(models.Model): ) 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 filtered_args = [] @@ -39,25 +35,25 @@ class ResPartner(models.Model): if isinstance(arg, (list, tuple)) and len(arg) == 3: # Skip active filters if arg[0] == "active": - _logger.info(f" Removing active filter: {arg}") + _logger.debug(f"Removing active filter: {arg}") 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: - _logger.info(f" Removing partner_share filter: {arg}") + _logger.debug(f"Removing partner_share filter: {arg}") continue filtered_args.append(arg) args = filtered_args 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) if not count: 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: - _logger.info(f"First 5 result IDs: {result_ids[:5]}") + _logger.debug(f"First result IDs: {result_ids[:5]}") return result \ No newline at end of file