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>
This commit is contained in:
parent
75de1ab999
commit
4a13bd7583
2 changed files with 5 additions and 13 deletions
|
|
@ -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
|
||||
|
|
|
|||
|
|
@ -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>
|
||||
|
|
|
|||
Reference in a new issue