Merge pull request #21666 from abpframework/DefaultAbpRequestLocalizationOptionsProvider

Ignore `defaultLanguage` if there is only one language.

Author: EngincanV

framework/src/Volo.Abp.AspNetCore/Microsoft/AspNetCore/RequestLocalization/DefaultAbpRequestLocalizationOptionsProvider.cs

@@ -4,6 +4,7 @@
 using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
+using DeviceDetectorNET;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Localization;
 using Microsoft.Extensions.DependencyInjection;
@@ -57,7 +58,7 @@ public virtual async Task<RequestLocalizationOptions> GetLocalizationOptionsAsyn
                             ? new RequestLocalizationOptions()
                             : new RequestLocalizationOptions
                             {
-                                DefaultRequestCulture = DefaultGetRequestCulture(defaultLanguage, languages),
+                                DefaultRequestCulture = GetDefaultRequestCulture(defaultLanguage, languages),
                                 SupportedCultures = languages
                                     .Select(l => l.CultureName)
                                     .Distinct()
@@ -87,15 +88,22 @@ public virtual async Task<RequestLocalizationOptions> GetLocalizationOptionsAsyn
         return _requestLocalizationOptions;
     }
 
-    private static RequestCulture DefaultGetRequestCulture(string? defaultLanguage, IReadOnlyList<LanguageInfo> languages)
+    private static RequestCulture GetDefaultRequestCulture(string? defaultLanguage, IReadOnlyList<LanguageInfo> languages)
     {
         if (defaultLanguage == null)
         {
-            var firstLanguage = languages.First();
+            var firstLanguage = languages.FirstOrDefault() ?? new LanguageInfo("en", "en");
             return new RequestCulture(firstLanguage.CultureName, firstLanguage.UiCultureName);
         }
 
         var (cultureName, uiCultureName) = LocalizationSettingHelper.ParseLanguageSetting(defaultLanguage);
+
+        if (languages.Any() && languages.All(l => l.CultureName != cultureName))
+        {
+            var firstLanguage = languages.First();
+            return new RequestCulture(firstLanguage.CultureName, firstLanguage.UiCultureName);
+        }
+
         return new RequestCulture(cultureName, uiCultureName);
     }
 
@@ -106,4 +114,4 @@ public virtual async Task HandleEventAsync(LanguageChangedEto eventData)
             _requestLocalizationOptions = null;
         }
     }
-}
+}

framework/src/Volo.Abp.Localization/Volo/Abp/Localization/LocalizationSettingHelper.cs

@@ -8,17 +8,25 @@ public static class LocalizationSettingHelper
     /// Gets a setting value like "en-US;en" and returns as splitted values like ("en-US", "en").
     /// </summary>
     /// <param name="settingValue"></param>
+    /// <param name="defaultCultureName"></param>
     /// <returns></returns>
-    public static (string cultureName, string uiCultureName) ParseLanguageSetting([NotNull] string settingValue)
+    public static (string cultureName, string uiCultureName) ParseLanguageSetting([NotNull] string settingValue, string defaultCultureName = "en")
     {
         Check.NotNull(settingValue, nameof(settingValue));
 
         if (!settingValue.Contains(";"))
         {
-            return (settingValue, settingValue);
+            return CultureHelper.IsValidCultureCode(settingValue)
+                ? (settingValue, settingValue)
+                : (defaultCultureName, defaultCultureName);
         }
 
         var splitted = settingValue.Split(';');
-        return (splitted[0], splitted[1]);
+        if (splitted.Length == 2 && CultureHelper.IsValidCultureCode(splitted[0]) && CultureHelper.IsValidCultureCode(splitted[1]))
+        {
+            return (splitted[0], splitted[1]);
+        }
+
+        return (defaultCultureName, defaultCultureName);
     }
 }