TvdbCultureInfo back to static class

jellyfin · Mar 23, 2024 · 5f98ffd · 5f98ffd
1 parent d15f92a
commit 5f98ffd
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 14 deletions.
diff --git a/Jellyfin.Plugin.Tvdb/TvdbClientManager.cs b/Jellyfin.Plugin.Tvdb/TvdbClientManager.cs
@@ -46,8 +46,9 @@ public TvdbClientManager(IApplicationHost applicationHost, ILocalizationManager
 
         _tokenUpdatedAt = DateTime.MinValue;
 
-        // Creates a new instance of the TvdbCultureInfo class to popluate the _cultures and _countries fields.
-        _ = new TvdbCultureInfo(localizationManager);
+        // Set the cultures and countries for the TvdbCultureInfo
+        TvdbCultureInfo.SetCultures(localizationManager.GetCultures());
+        TvdbCultureInfo.SetCountries(localizationManager.GetCountries());
     }
 
     private static string? UserPin => TvdbPlugin.Instance?.Configuration.SubscriberPIN;

diff --git a/Jellyfin.Plugin.Tvdb/TvdbCultureInfo.cs b/Jellyfin.Plugin.Tvdb/TvdbCultureInfo.cs
@@ -9,29 +9,27 @@ namespace Jellyfin.Plugin.Tvdb
     /// <summary>
     /// Tvdb culture info.
     /// </summary>
-    public class TvdbCultureInfo
+    internal static class TvdbCultureInfo
     {
         private static IEnumerable<CultureDto> _cultures = new List<CultureDto>();
         private static IEnumerable<CountryInfo> _countries = new List<CountryInfo>();
-        private readonly ILocalizationManager _localizationManager;
 
-        /// <summary>
-        /// Initializes a new instance of the <see cref="TvdbCultureInfo"/> class.
-        /// </summary>
-        /// <param name="localizationManager">Instance of the <see cref="ILocalizationManager"/> interface.</param>
-        public TvdbCultureInfo(ILocalizationManager localizationManager)
+        internal static void SetCultures(IEnumerable<CultureDto> cultures)
+        {
+            _cultures = cultures;
+        }
+
+        internal static void SetCountries(IEnumerable<CountryInfo> countries)
         {
-            _localizationManager = localizationManager;
-            _countries = _localizationManager.GetCountries();
-            _cultures = _localizationManager.GetCultures();
+            _countries = countries;
         }
 
         /// <summary>
         /// Gets the cultureinfo for the given language.
         /// </summary>
         /// <param name="language">Language.</param>
         /// <returns>CultureInfo.</returns>
-        public static CultureDto? GetCultureInfo(string language)
+        internal static CultureDto? GetCultureInfo(string language)
         {
             foreach (var culture in _cultures)
             {
@@ -52,7 +50,7 @@ public TvdbCultureInfo(ILocalizationManager localizationManager)
         /// </summary>
         /// <param name="country"> Country.</param>
         /// <returns>CountryInfo.</returns>
-        public static CountryInfo? GetCountryInfo(string country)
+        internal static CountryInfo? GetCountryInfo(string country)
         {
             foreach (var countryInfo in _countries)
             {