From cae448a9c682984cf06f8e3d120f2f17ba7f67f0 Mon Sep 17 00:00:00 2001
From: Lizandro Guarnizo <77708265+lizandrogd@users.noreply.github.com>
Date: Tue, 30 Jun 2026 19:02:08 -0500
Subject: [PATCH] fix: use $menus after it is defined (was undefined in
 welcomeMenuOpts loop)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 admin/DashboardController.php | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/admin/DashboardController.php b/admin/DashboardController.php
index b6d86b2..b7dfca2 100644
--- a/admin/DashboardController.php
+++ b/admin/DashboardController.php
@@ -2875,14 +2875,14 @@ HTML;
         $approvalWebhookVal = self::h($config['approval_webhook'] ?? '');
         $ignoreVal = self::h(implode("\n", $config['ignore_prefixes'] ?? []));
         $welcomeMenuVal = self::h($config['welcome_menu'] ?? '');
+        $commands = $config['commands'] ?? [];
+        $menus = $config['menus'] ?? [];
+        $flows = $config['flows'] ?? [];
         $welcomeMenuOpts = '';
         foreach (array_keys($menus) as $mk) {
             $s = ($config['welcome_menu'] ?? '') === $mk ? ' selected' : '';
             $welcomeMenuOpts .= '<option value="' . self::h($mk) . '"' . $s . '>' . self::h($mk) . '</option>';
         }
-        $commands = $config['commands'] ?? [];
-        $menus = $config['menus'] ?? [];
-        $flows = $config['flows'] ?? [];
         $aiProviderOpenai  = $aiProviderVal === 'openai'  ? ' selected' : '';
         $aiProviderGemini  = $aiProviderVal === 'gemini'  ? ' selected' : '';
         $aiProviderMock    = $aiProviderVal === 'mock'    ? ' selected' : '';