payfrit/payfrit-works
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert deduplication hack - root cause fixed in app

Browse source 
The duplicate modifiers were caused by the app creating nested groups.
That's now fixed, so we can remove the deduplication workaround.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
John Mizerek 2026-01-13 22:52:12 -08:00
parent f6c3729874
commit a6259bccb0
1 changed files with 0 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
kds/kds.js
View file

@ -361,20 +361,12 @@ function renderAllModifiers(modifiers, allItems) {
collectLeafModifiers(modifiers); collectLeafModifiers(modifiers);
console.log(` Total leaf modifiers found: ${leafModifiers.length}`); console.log(` Total leaf modifiers found: ${leafModifiers.length}`);
// Deduplicate modifiers with the same display text
const seen = new Set();
leafModifiers.forEach(({ mod }) => { leafModifiers.forEach(({ mod }) => {
// Use ItemParentName (the category/template name) if available, otherwise just show the item name // Use ItemParentName (the category/template name) if available, otherwise just show the item name
// This gives us "Drink Choice: Coke" instead of "Double Double Combo: Coke" // This gives us "Drink Choice: Coke" instead of "Double Double Combo: Coke"
const displayText = mod.ItemParentName const displayText = mod.ItemParentName
? `${mod.ItemParentName}: ${mod.ItemName}` ? `${mod.ItemParentName}: ${mod.ItemName}`
: mod.ItemName; : mod.ItemName;
// Skip duplicates
if (seen.has(displayText)) return;
seen.add(displayText);
html += `<div class="modifier">+ ${escapeHtml(displayText)}</div>`; html += `<div class="modifier">+ ${escapeHtml(displayText)}</div>`;
}); });