From 34b3c72631c898c4d1b670c618d1f45d14f04b57 Mon Sep 17 00:00:00 2001
From: Daniel Palumbo <dpalumbo06@gmail.com>
Date: Mon, 3 Aug 2026 11:32:36 +0200
Subject: [PATCH 1/2] [T1523] ADD: Preview PDF button on communication job

The existing Preview button only rendered raw HTML, never the actual PDF
report, so a broken image (a positioning/CSS bug, or a static asset that
fails to load) could go unnoticed until physical printing. Add a Preview PDF
button that renders through the real ir.actions.report action instead.
---
 partner_communication/models/communication_job.py      | 6 ++++++
 partner_communication/views/communication_job_view.xml | 8 ++++++++
 2 files changed, 14 insertions(+)

diff --git a/partner_communication/models/communication_job.py b/partner_communication/models/communication_job.py
index ab5da275e..1cc613822 100644
--- a/partner_communication/models/communication_job.py
+++ b/partner_communication/models/communication_job.py
@@ -816,6 +816,12 @@ def preview_pdf(self):
             "target": "new",
         }
 
+    def preview_pdf_report(self):
+        self.ensure_one()
+        return self.report_id.with_context(lang=self.partner_id.lang).report_action(
+            self.ids, config=False
+        )
+
     def download_data(self):
         action = {
             "type": "ir.actions.act_window",
diff --git a/partner_communication/views/communication_job_view.xml b/partner_communication/views/communication_job_view.xml
index 29f4fbe7e..debddd77f 100644
--- a/partner_communication/views/communication_job_view.xml
+++ b/partner_communication/views/communication_job_view.xml
@@ -82,6 +82,14 @@
               icon="fa-search-plus"
               class="oe_stat_button"
               invisible="state != 'pending'"
+            />
+                        <button
+              name="preview_pdf_report"
+              string="Preview PDF"
+              type="object"
+              icon="fa-file-pdf-o"
+              class="oe_stat_button"
+              invisible="state != 'pending' or not report_id"
             />
                         <button
               name="open_related"

From dc4353e59817e18ba2101dfe7c3864ca09f11858 Mon Sep 17 00:00:00 2001
From: Daniel Palumbo <dpalumbo06@gmail.com>
Date: Mon, 3 Aug 2026 15:21:03 +0200
Subject: [PATCH 2/2] [T1523] FIX: Preview PDF must not reach the physical
 printer

preview_pdf_report() called report_action() without
must_skip_send_to_printer=True, so previewing a report configured through
base_report_to_printer for physical printing produced a real printer job
instead of just serving the PDF to the browser.
---
 partner_communication/models/communication_job.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/partner_communication/models/communication_job.py b/partner_communication/models/communication_job.py
index 1cc613822..3b6ec3afa 100644
--- a/partner_communication/models/communication_job.py
+++ b/partner_communication/models/communication_job.py
@@ -818,9 +818,9 @@ def preview_pdf(self):
 
     def preview_pdf_report(self):
         self.ensure_one()
-        return self.report_id.with_context(lang=self.partner_id.lang).report_action(
-            self.ids, config=False
-        )
+        return self.report_id.with_context(
+            lang=self.partner_id.lang, must_skip_send_to_printer=True
+        ).report_action(self.ids, config=False)
 
     def download_data(self):
         action = {