From aa447bd009509323385f853486de0133e58b0dd5 Mon Sep 17 00:00:00 2001
From: John Mizerek <mizerek@gmail.com>
Date: Fri, 13 Feb 2026 10:32:18 -0800
Subject: [PATCH] Fix extractDir path detection for ZIP scanning

- Extract UUID folder path from URL instead of using getDirectoryFromPath
- Old logic was broken: listLast on path ending with / returned empty string
- This caused the code to go up one level too far

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 api/setup/analyzeMenuUrl.cfm | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/api/setup/analyzeMenuUrl.cfm b/api/setup/analyzeMenuUrl.cfm
index 41d8bce..800b6dc 100644
--- a/api/setup/analyzeMenuUrl.cfm
+++ b/api/setup/analyzeMenuUrl.cfm
@@ -387,11 +387,9 @@
                     </cfif>
 
                     <!--- Scan ALL HTML files in the ZIP for business info --->
-                    <cfset extractDir = getDirectoryFromPath(localFilePath)>
-                    <!--- Go up one level if we're in a subfolder (e.g., Menu_files) --->
-                    <cfif NOT findNoCase("menu-import", listLast(extractDir, "/\"))>
-                        <cfset extractDir = getDirectoryFromPath(left(extractDir, len(extractDir) - 1))>
-                    </cfif>
+                    <!--- Get the extraction directory from the URL (the UUID folder) --->
+                    <cfset extractDir = reReplaceNoCase(targetUrl, "https?://[^/]+(/temp/menu-import/[a-f0-9]+/).*", "\1")>
+                    <cfset extractDir = expandPath(extractDir)>
                     <cftry>
                         <cfdirectory action="list" directory="#extractDir#" name="allHtmlFiles" filter="*.htm*" recurse="true" type="file">
                         <cfset arrayAppend(response.steps, "Found " & allHtmlFiles.recordCount & " HTML files in ZIP")>