From 4c8226eb18c6278bd1d0d1830c105339d3992aa8 Mon Sep 17 00:00:00 2001
From: khushi8112 <khushirawat23@navgurukul.org>
Date: Fri, 7 Nov 2025 12:55:14 +0530
Subject: [PATCH] fix: show company-details popup only for the targeted print
 format/letterhead

---
 erpnext/public/js/print.js | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/erpnext/public/js/print.js b/erpnext/public/js/print.js
index 56bb25b48d7..53f03f83bda 100644
--- a/erpnext/public/js/print.js
+++ b/erpnext/public/js/print.js
@@ -1,6 +1,16 @@
 let beforePrintHandled = false;
 
 frappe.realtime.on("sales_invoice_before_print", (data) => {
+	let print_format = $('input[data-fieldname="print_format"]').val();
+	let letterhead = $('input[data-fieldname="letterhead"]').val();
+
+	let allowed_print_formats = ["Sales Invoice Standard", "Sales Invoice with Item Image"];
+	let allowed_letterheads = ["Company Letterhead", "Company Letterhead - Grey"];
+
+	if (!allowed_print_formats.includes(print_format) && !allowed_letterheads.includes(letterhead)) {
+		return;
+	}
+
 	const route = frappe.get_route();
 
 	if (!beforePrintHandled && route[0] === "print" && route[1] === "Sales Invoice") {