From 0038c481abd4b0eca6a9f64d8df208db87faa4b6 Mon Sep 17 00:00:00 2001
From: John Mizerek <mizerek@gmail.com>
Date: Sun, 18 Jan 2026 12:26:36 -0800
Subject: [PATCH] Fix brand color modal cancel button and simplify API query

- Changed hideModal() to closeModal() in color picker
- Simplified queryExecute parameter handling

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 api/businesses/saveBrandColor.cfm | 2 +-
 portal/menu-builder.html          | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/api/businesses/saveBrandColor.cfm b/api/businesses/saveBrandColor.cfm
index 4dd641b..a31c2ae 100644
--- a/api/businesses/saveBrandColor.cfm
+++ b/api/businesses/saveBrandColor.cfm
@@ -52,7 +52,7 @@ queryExecute("
   SET BusinessBrandColor = :color
   WHERE BusinessID = :bizId
 ", {
-  color: { value: len(brandColor) ? brandColor : "", null: !len(brandColor), cfsqltype: "cf_sql_varchar" },
+  color: { value: brandColor, cfsqltype: "cf_sql_varchar" },
   bizId: { value: bizId, cfsqltype: "cf_sql_integer" }
 }, { datasource: "payfrit" });
 
diff --git a/portal/menu-builder.html b/portal/menu-builder.html
index 16ff966..3e97bc5 100644
--- a/portal/menu-builder.html
+++ b/portal/menu-builder.html
@@ -3022,7 +3022,7 @@
             <button type="button" class="color-preset" data-color="#355E3B" style="width: 32px; height: 32px; border-radius: 4px; border: 2px solid transparent; cursor: pointer; background: #355E3B;" title="Hunter Green"></button>
           </div>
           <div style="display: flex; gap: 8px; justify-content: flex-end;">
-            <button type="button" class="btn btn-secondary" onclick="MenuBuilder.hideModal()">Cancel</button>
+            <button type="button" class="btn btn-secondary" onclick="MenuBuilder.closeModal()">Cancel</button>
             <button type="button" class="btn btn-primary" onclick="MenuBuilder.saveBrandColor()">Save Color</button>
           </div>
         `;
@@ -3080,7 +3080,7 @@
           if (data.OK) {
             this.brandColor = color;
             document.getElementById('brandColorSwatch').style.background = color;
-            this.hideModal();
+            this.closeModal();
             this.toast('Brand color saved!', 'success');
           } else {
             this.toast(data.MESSAGE || 'Failed to save color', 'error');