From b441250770547357a0e89471be870ff6ab4669f9 Mon Sep 17 00:00:00 2001 From: Kristof Bernaert Date: Mon, 26 Jan 2026 02:02:58 +0100 Subject: [PATCH] Fix allow_mail_archived_partner causing system notifications to send emails --- .../models/res_partner.py | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/allow_mail_archived_partner/models/res_partner.py b/allow_mail_archived_partner/models/res_partner.py index 10feae1..7cbca99 100755 --- a/allow_mail_archived_partner/models/res_partner.py +++ b/allow_mail_archived_partner/models/res_partner.py @@ -10,13 +10,12 @@ class ResPartner(models.Model): @api.model def _search(self, args, offset=0, limit=None, order=None, count=False, access_rights_uid=None): """ - DEBUG search method + DEBUG search method - FIXED for Odoo 16 Query object """ _logger.info("=== RES.PARTNER _search ===") _logger.info(f"Search args before: {args}") _logger.info(f"Context include_archived: {self.env.context.get('include_archived_partners')}") _logger.info(f"Context active_test: {self.env.context.get('active_test')}") - _logger.info(f"Context mail_notify_force: {self.env.context.get('mail_notify_force')}") if self.env.context.get("include_archived_partners") or self.env.context.get("mail_notify_force"): _logger.info("Context flags found - removing active filters") @@ -32,6 +31,17 @@ class ResPartner(models.Model): self = self.with_context(active_test=False) _logger.info(f"Search args after: {args}") + # Call parent result = super()._search(args, offset, limit, order, count, access_rights_uid) - _logger.info(f"Search result: {result[:10] if not count else 'count='+str(result)}") + + # FIX: Don't try to slice Query object + if not count: + # Convert to list to check results (only for debugging) + result_ids = list(result) + _logger.info(f"Search returned {len(result_ids)} results") + if result_ids: + _logger.info(f"First few result IDs: {result_ids[:10]}") + else: + _logger.info(f"Search count result: {result}") + return result \ No newline at end of file