Merge pull request #7035 from ProcessMaker/bugfix/FOUR-17008

FOUR-17008: [next] Define a value when the config is Email Server 1, 2
ProcessMaker · Jul 24, 2024 · 4b6728d · 4b6728d
2 parents 6c5140f + ea2b56b
commit 4b6728d
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/ProcessMaker/Models/Setting.php b/ProcessMaker/Models/Setting.php
@@ -430,6 +430,7 @@ function ($settings) use ($id) {
     public static function updateAllSettingsGroupId()
     {
         Setting::whereNull('group_id')->chunk(100, function ($settings) {
+            $defaultId = SettingsMenus::EMAIL_MENU_GROUP;
             foreach ($settings as $setting) {
                 // Define the value of 'menu_group' based on 'group'
                 switch ($setting->group) {
@@ -466,7 +467,11 @@ public static function updateAllSettingsGroupId()
                         $id = null;
                         break;
                     default: // The default value
-                        $id = SettingsMenus::getId(SettingsMenus::EMAIL_MENU_GROUP);
+                        if (preg_match('/^Email Server/', $setting->group)) {
+                            $id = SettingsMenus::getId(SettingsMenus::EMAIL_MENU_GROUP);
+                        } else {
+                            $id = SettingsMenus::getId($defaultId);
+                        }
                         break;
                 }
                 if ($id !== null) {