fix(AdminSettings/AI): show pref list of only the enabled translation providers

Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>

diff --git a/apps/settings/lib/Settings/Admin/ArtificialIntelligence.php b/apps/settings/lib/Settings/Admin/ArtificialIntelligence.php
index 41b9e837c7e1ca732779101fe9a6839b1cf93ec6..ec266f70987848ef18e02b35c95f8932ba4986ae 100644 (file)
--- a/apps/settings/lib/Settings/Admin/ArtificialIntelligence.php
+++ b/apps/settings/lib/Settings/Admin/ArtificialIntelligence.php
@@ -138,7 +138,12 @@ class ArtificialIntelligence implements IDelegatedSettings {
                                                $value = array_merge($defaultValue, $value);
                                                break;
                                        case 'ai.translation_provider_preferences':
-                                               $value += array_diff($defaultValue, $value); // Add entries from $defaultValue that are not in $value to the end of $value
+                                               // Only show entries from $value (saved pref list) that are in $defaultValue (enabled providers)
+                                               // and add all providers that are enabled but not in the pref list
+                                               if (!is_array($defaultValue)) {
+                                                       break;
+                                               }
+                                               $value = array_values(array_unique(array_merge(array_intersect($value, $defaultValue), $defaultValue), SORT_STRING));
                                                break;
                                        default:
                                                break;