Merge pull request #38 from intility/remove-or-max-count

refactor: 🔨 remove the OR max count

Author: erlendellefsen

JsonApiToolkit.Tests/Extensions/IncludeFilterParserTests.cs

@@ -236,39 +236,6 @@ public void SeparateIncludeFilters_WithFilterOnNonIncludedRelationship_ReturnsAs
         Assert.Empty(includeFilters);
     }
 
-    [Fact]
-    public void SeparateIncludeFilters_WithTooManyOrConditions_ThrowsException()
-    {
-        // Arrange
-        var filters = new FilterGroup
-        {
-            LogicalOperator = LogicalOperator.Or,
-            Filters = new List<FilterParameter>(),
-        };
-
-        // Add 11 OR conditions (exceeds limit of 10)
-        for (int i = 0; i < 11; i++)
-        {
-            filters.Filters.Add(
-                new FilterParameter
-                {
-                    Field = "comments.status",
-                    Operator = FilterOperator.Eq,
-                    Value = $"value{i}",
-                }
-            );
-        }
-
-        var includePaths = new List<string> { "comments" };
-
-        // Act & Assert
-        var exception = Assert.Throws<JsonApiBadRequestException>(
-            () => IncludeFilterParser.SeparateIncludeFilters(filters, includePaths)
-        );
-
-        Assert.Contains("Too many OR conditions", exception.Message);
-    }
-
     [Fact]
     public void SeparateIncludeFilters_WithTooDeepNesting_ThrowsException()
     {

JsonApiToolkit/Extensions/Querying/Filtering/IncludeFilterParser.cs

@@ -10,7 +10,6 @@ public static class IncludeFilterParser
 {
     private const int MaxIncludeFilterDepth = 3;
     private const int MaxIncludeFilters = 20;
-    private const int MaxOrConditions = 10;
 
     /// <summary>
     /// Separates main filters from include filters.
@@ -57,14 +56,6 @@ Dictionary<string, FilterGroup> filtersByRelationship
     {
         var newGroup = new FilterGroup { LogicalOperator = group.LogicalOperator };
 
-        // Check OR conditions count
-        if (group.LogicalOperator == LogicalOperator.Or && group.Filters.Count > MaxOrConditions)
-        {
-            throw new JsonApiBadRequestException(
-                $"Too many OR conditions in filter group. Maximum allowed: {MaxOrConditions}"
-            );
-        }
-
         // Track filters for each relationship in this group
         var localIncludeFilters = new Dictionary<string, FilterGroup>(StringComparer.OrdinalIgnoreCase);