Apply review suggestion

jellyfin · Feb 19, 2023 · a1c9fd9 · a1c9fd9
1 parent 6b24798
commit a1c9fd9
Showing 1 changed file with 20 additions and 8 deletions.
diff --git a/MediaBrowser.Model/Dlna/ConditionProcessor.cs b/MediaBrowser.Model/Dlna/ConditionProcessor.cs
@@ -140,10 +140,16 @@ private static bool IsConditionSatisfied(ProfileCondition condition, int? curren
             var conditionType = condition.Condition;
             if (condition.Condition == ProfileConditionType.EqualsAny)
             {
-                return condition.Value.Split('|')
-                    .Select(s => int.TryParse(s, NumberStyles.Any, CultureInfo.InvariantCulture, out int n) ? n : (int?)null)
-                    .Where(n => n is not null)
-                    .Contains(currentValue.Value);
+                foreach (var singleConditionString in condition.Value.AsSpan().Split('|'))
+                {
+                    if (int.TryParse(singleConditionString, NumberStyles.Any, CultureInfo.InvariantCulture, out int conditionValue)
+                        && conditionValue.Equals(currentValue))
+                    {
+                        return true;
+                    }
+                }
+
+                return false;
             }
 
             if (int.TryParse(condition.Value, NumberStyles.Any, CultureInfo.InvariantCulture, out var expected))
@@ -224,10 +230,16 @@ private static bool IsConditionSatisfied(ProfileCondition condition, double? cur
             var conditionType = condition.Condition;
             if (condition.Condition == ProfileConditionType.EqualsAny)
             {
-                return condition.Value.Split('|')
-                    .Select(s => double.TryParse(s, NumberStyles.Any, CultureInfo.InvariantCulture, out double n) ? n : (double?)null)
-                    .Where(n => n is not null)
-                    .Contains(currentValue.Value);
+                foreach (var singleConditionString in condition.Value.AsSpan().Split('|'))
+                {
+                    if (double.TryParse(singleConditionString, NumberStyles.Any, CultureInfo.InvariantCulture, out double conditionValue)
+                        && conditionValue.Equals(currentValue))
+                    {
+                        return true;
+                    }
+                }
+
+                return false;
             }
 
             if (double.TryParse(condition.Value, NumberStyles.Any, CultureInfo.InvariantCulture, out var expected))