From 9a280916013d31bd48250cb5204ec695d5509c52 Mon Sep 17 00:00:00 2001
From: John Mizerek <mizerek@gmail.com>
Date: Mon, 9 Mar 2026 21:04:34 -0700
Subject: [PATCH] Handle struct types in item category and modifiers during
 merge

Claude sometimes returns category or modifiers as objects instead
of strings. Added isSimpleValue checks to prevent cast errors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 api/setup/analyzeMenuImages.cfm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/api/setup/analyzeMenuImages.cfm b/api/setup/analyzeMenuImages.cfm
index f915094..8c53902 100644
--- a/api/setup/analyzeMenuImages.cfm
+++ b/api/setup/analyzeMenuImages.cfm
@@ -401,7 +401,8 @@
         </cfif>
 
         <!--- Get item's category --->
-        <cfset itemCategory = structKeyExists(item, "category") ? trim(item.category) : "">
+        <cfset itemCatRaw = structKeyExists(item, "category") ? item.category : "">
+        <cfset itemCategory = isSimpleValue(itemCatRaw) ? trim(itemCatRaw) : "">
 
         <!--- Find category-level modifiers for this category --->
         <cfif len(itemCategory)>
@@ -412,7 +413,8 @@
                         <!--- Check if modifier is not already assigned --->
                         <cfset alreadyAssigned = false>
                         <cfloop array="#item.modifiers#" index="existingMod">
-                            <cfif lCase(existingMod) EQ lCase(mod.name)>
+                            <cfset existingModName = isSimpleValue(existingMod) ? existingMod : (isStruct(existingMod) AND structKeyExists(existingMod, "name") ? existingMod.name : "")>
+                            <cfif len(existingModName) AND lCase(existingModName) EQ lCase(mod.name)>
                                 <cfset alreadyAssigned = true>
                                 <cfbreak>
                             </cfif>