From 4a13bd7583b1adcfd61c3044b993a1369ebd4d0a Mon Sep 17 00:00:00 2001
From: John Mizerek <mizerek@gmail.com>
Date: Wed, 11 Mar 2026 17:40:42 -0700
Subject: [PATCH] Remove Inverted Display toggle; fix template modifier name
 not saving

- Removed IsInvertedGroup toggle and badge from menu builder UI
- Fixed saveFromBuilder.cfm not updating Name for template modifiers
  (only selection rules were saved, name changes were silently lost)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 api/menu/saveFromBuilder.cfm |  6 ++++--
 portal/menu-builder.html     | 12 +-----------
 2 files changed, 5 insertions(+), 13 deletions(-)

diff --git a/api/menu/saveFromBuilder.cfm b/api/menu/saveFromBuilder.cfm
index 22eae40..5d82628 100644
--- a/api/menu/saveFromBuilder.cfm
+++ b/api/menu/saveFromBuilder.cfm
@@ -333,15 +333,17 @@ try {
                                     sortOrder: modSortOrder
                                 });
 
-                                // Update template's selection rules
+                                // Update template's name and selection rules
                                 queryTimed("
                                     UPDATE Items
-                                    SET RequiresChildSelection = :requiresSelection,
+                                    SET Name = :name,
+                                        RequiresChildSelection = :requiresSelection,
                                         MaxNumSelectionReq = :maxSelections,
                                         IsInvertedGroup = :isInverted
                                     WHERE ID = :modID
                                 ", {
                                     modID: modDbId,
+                                    name: mod.name,
                                     requiresSelection: requiresSelection,
                                     maxSelections: maxSelections,
                                     isInverted: (structKeyExists(mod, "isInverted") && mod.isInverted) ? 1 : 0
diff --git a/portal/menu-builder.html b/portal/menu-builder.html
index 923ed2f..c22e07e 100644
--- a/portal/menu-builder.html
+++ b/portal/menu-builder.html
@@ -2245,16 +2245,6 @@
             <p style="color: var(--gray-500); font-size: 11px; margin: 4px 0 0;">
               Required = customer must select at least one option. Max 0 = unlimited.
             </p>
-            <div class="property-group" style="margin-top: 12px;">
-              <label>Inverted Display</label>
-              <select onchange="MenuBuilder.updateOption('${parent.id}', '${option.id}', 'isInverted', this.value === 'true')">
-                <option value="false" ${!option.isInverted ? 'selected' : ''}>No</option>
-                <option value="true" ${option.isInverted ? 'selected' : ''}>Yes</option>
-              </select>
-              <p style="color: var(--gray-500); font-size: 11px; margin: 4px 0 0;">
-                When enabled, KDS and cart only show removed items (e.g., "NO Mustard") instead of listing all defaults.
-              </p>
-            </div>
           </div>
           ` : ''}
           <div class="property-group">
@@ -4247,7 +4237,7 @@
                   ${mod.isDefault ? '<span class="item-type-badge">Default</span>' : ''}
                   ${mod.requiresSelection ? '<span class="item-type-badge" style="background: #dc3545; color: white;">Required</span>' : ''}
                   ${mod.maxSelections > 0 ? `<span class="item-type-badge">Max ${mod.maxSelections}</span>` : ''}
-                  ${mod.isInverted ? '<span class="item-type-badge" style="background: #6f42c1; color: white;">Inverted</span>' : ''}
+
                   ${hasOptions ? `<span class="item-type-badge">${mod.options.length} options</span>` : ''}
                 </div>
               </div>