Bladeren bron

#182 - Eliminate duplicates in the selection list for max rating in user screen

Luke Pulverenti 12 jaren geleden
bovenliggende
commit
47d2a11bdf

+ 7 - 2
MediaBrowser.Controller/Entities/BaseItem.cs

@@ -849,10 +849,15 @@ namespace MediaBrowser.Controller.Entities
         {
             if (user == null)
             {
-                throw new ArgumentNullException();
+                throw new ArgumentNullException("user");
+            }
+
+            if (string.IsNullOrEmpty(user.Configuration.MaxParentalRating))
+            {
+                return true;
             }
 
-            return user.Configuration.MaxParentalRating == null || Ratings.Level(CustomRating ?? OfficialRating) <= user.Configuration.MaxParentalRating;
+            return Ratings.Level(CustomRating ?? OfficialRating) <= Ratings.Level(user.Configuration.MaxParentalRating);
         }
 
         /// <summary>

+ 2 - 1
MediaBrowser.Controller/Localization/Ratings.cs

@@ -1,4 +1,5 @@
-using MediaBrowser.Controller.Configuration;
+using System.Globalization;
+using MediaBrowser.Controller.Configuration;
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;

+ 1 - 1
MediaBrowser.Model/Configuration/UserConfiguration.cs

@@ -10,7 +10,7 @@ namespace MediaBrowser.Model.Configuration
         /// Gets or sets the max parental rating.
         /// </summary>
         /// <value>The max parental rating.</value>
-        public int? MaxParentalRating { get; set; }
+        public string MaxParentalRating { get; set; }
 
         /// <summary>
         /// Gets or sets a value indicating whether [use custom library].