From 14caad857d5feaa6fe5e5c4c18d6020824e595fc Mon Sep 17 00:00:00 2001
From: cproudlock <cproudlock@localhost>
Date: Mon, 8 Dec 2025 12:39:34 -0500
Subject: [PATCH] Fix printer selection panel showing N/A when CSF name is
 empty
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Use location (machine number/alias) as display name fallback when
printercsfname is not set. Show model number as secondary info instead
of location to avoid redundancy.

Fixes Coaching 115 and similar printers without CSF names assigned.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 printer_installer_map.asp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/printer_installer_map.asp b/printer_installer_map.asp
index 8d9189c..9506964 100644
--- a/printer_installer_map.asp
+++ b/printer_installer_map.asp
@@ -406,10 +406,12 @@ function updateSelectionUI() {
         var html = '';
         for (var printerId in selectedPrinters) {
             var printer = printerData[printerId];
+            // Use CSF name if available, otherwise fall back to location
+            var displayName = (printer.csf && printer.csf !== '') ? printer.csf : printer.location;
             html += '<div class="selected-printer-item">' +
                     '<span class="remove-btn" onclick="togglePrinter(\'' + printerId + '\')">×</span>' +
-                    '<strong>' + (printer.csf || 'N/A') + '</strong><br>' +
-                    '<span style="font-size:11px; color:#aaa;">' + printer.location + '</span>' +
+                    '<strong>' + displayName + '</strong><br>' +
+                    '<span style="font-size:11px; color:#aaa;">' + printer.model + '</span>' +
                     '</div>';
         }
         container.innerHTML = html;