Skip to content

Commit 4b989db

Browse files
committed
Removing ambiguous calls between versions of the framework
1 parent 9f224d8 commit 4b989db

File tree

7 files changed

+18
-15
lines changed

7 files changed

+18
-15
lines changed

src/Deveel.Filter.Model/Filters/Filter.cs

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -164,12 +164,15 @@ public static BinaryFilter LessThanOrEqual(Filter left, Filter right)
164164
public static BinaryFilter LessThanOrEqual(IFilter left, IFilter right)
165165
=> Binary(Convert(left), Convert(right), FilterType.LessThanOrEqual);
166166

167-
public static FunctionFilter Function(VariableFilter variable, string functionName, params Filter[] arguments)
167+
public static FunctionFilter Function(VariableFilter variable, string functionName, Filter[] arguments)
168168
=> new FunctionFilter(variable, functionName, arguments);
169169

170-
public static FunctionFilter Function(VariableFilter variable, string functionName, params IFilter[] arguments)
170+
public static FunctionFilter Function(VariableFilter variable, string functionName, IFilter[] arguments)
171171
=> new FunctionFilter(variable, functionName, arguments?.Select(Convert).ToArray());
172172

173+
public static FunctionFilter Function(VariableFilter variable, string functionName)
174+
=> new FunctionFilter(variable, functionName, Array.Empty<Filter>());
175+
173176
public static ConstantFilter Constant(object? value)
174177
=> new ConstantFilter(value);
175178

@@ -183,7 +186,7 @@ public static VariableFilter Variable(string variableName) {
183186
return new VariableFilter(variableName);
184187
}
185188

186-
#endregion
189+
#endregion
187190

188191
#region EmptyFilter
189192

test/Deveel.Filter.Model.Web.XUnit/Filters/ModelFromFilterTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ public static void BuildVariableModel() {
298298

299299
[Fact]
300300
public static void BuildFunctionModel() {
301-
var function = Filter.Function(Filter.Variable("x"), "f", Filter.Constant(22), Filter.Constant(45));
301+
var function = Filter.Function(Filter.Variable("x"), "f", new[] { Filter.Constant(22), Filter.Constant(45) });
302302
var model = function.ToFilterModel();
303303
Assert.NotNull(model);
304304
Assert.NotNull(model.Function);

test/Deveel.Filter.Model.XUnit/Filters/FilterStringTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ public static void UnaryOfConstantToString(object value, FilterType filterType,
8585
public static void FunctionToString(string variable, string functionName, object arg, string expected) {
8686
var variableFilter = Filter.Variable(variable);
8787
var constantFilter = Filter.Constant(arg);
88-
var functionFilter = Filter.Function(variableFilter, functionName, constantFilter);
88+
var functionFilter = Filter.Function(variableFilter, functionName, new[] { constantFilter });
8989
var actual = functionFilter.ToString();
9090
Assert.Equal(expected, actual);
9191
}

test/Deveel.Filter.Model.XUnit/Filters/LambdaFunctionTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ public static void SimpleUnaryAsLambda() {
2121
[Fact]
2222
public static void ComplexBinaryWithFunctionCallAsLabda() {
2323
var filter = Filter.Binary(
24-
Filter.Function(Filter.Variable("x"), "Substring", Filter.Constant(1)),
24+
Filter.Function(Filter.Variable("x"), "Substring", new[] { Filter.Constant(1) }),
2525
Filter.Constant("oobar"),
2626
FilterType.Equal);
2727

@@ -31,7 +31,7 @@ public static void ComplexBinaryWithFunctionCallAsLabda() {
3131

3232
[Fact]
3333
public static void SimpleFunctionCallAslambda() {
34-
var filter = Filter.Function(Filter.Variable("x"), "StartsWith", Filter.Constant("foo"));
34+
var filter = Filter.Function(Filter.Variable("x"), "StartsWith", new[] { Filter.Constant("foo") });
3535

3636
var result = filter.Evaluate("x", "foobar");
3737
Assert.True(result);

test/Deveel.Filter.Model.XUnit/Filters/VisitorTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public static void VisitUnary() {
5252

5353
[Fact]
5454
public static void VisitFunction() {
55-
var function = Filter.Function(Filter.Variable("x"), "test", Filter.Constant(123));
55+
var function = Filter.Function(Filter.Variable("x"), "test", new[] { Filter.Constant(123) });
5656

5757
var visitor = new FilterVisitor();
5858
var result = visitor.Visit(function);

test/Deveel.Filter.MongoBson.XUnit/Filters/BsonFilterSerializationTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ public static void SerializeNot() {
132132
[InlineData("x", "Contains", 123)]
133133
public static void SerializeFunction(string varName, string functionName, object arg) {
134134
var variable = Filter.Variable(varName);
135-
var filter = Filter.Function(variable, functionName, Filter.Constant(arg));
135+
var filter = Filter.Function(variable, functionName, new[] { Filter.Constant(arg) });
136136
var bson = filter.AsBsonDocument();
137137
Assert.NotNull(bson);
138138
Assert.Equal("function", bson["type"].AsString);

test/Deveel.Filters.DynamicLinq.XUnit/Filters/DynamicFilterTests.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ public static void AsDynamicLambda_StringMethods_ShouldConvertToLambda()
289289
var filter = Filter.Function(
290290
Filter.Variable("Name"),
291291
"Contains",
292-
Filter.Constant("John"));
292+
new[] { Filter.Constant("John") });
293293

294294
// Act
295295
var lambda = filter.AsDynamicLambda<Person>();
@@ -434,7 +434,7 @@ public static void AsDynamicLamda_PrimitiveTypes_ShouldWork()
434434
Assert.False((bool)intCompiled.DynamicInvoke(3)!);
435435

436436
// Test with string
437-
var stringFilter = Filter.Function(Filter.Variable("x"), "StartsWith", Filter.Constant("Hello"));
437+
var stringFilter = Filter.Function(Filter.Variable("x"), "StartsWith", new[] { Filter.Constant("Hello") });
438438
var stringLambda = stringFilter.AsDynamicLamda(typeof(string));
439439
var stringCompiled = stringLambda.Compile();
440440
Assert.True((bool)stringCompiled.DynamicInvoke("Hello World")!);
@@ -701,7 +701,7 @@ public static void AsDynamicLambda_StringStartsWith_ShouldWork()
701701
var filter = Filter.Function(
702702
Filter.Variable("Name"),
703703
"StartsWith",
704-
Filter.Constant("J"));
704+
new[] { Filter.Constant("J") });
705705

706706
// Act
707707
var lambda = filter.AsDynamicLambda<Person>();
@@ -724,7 +724,7 @@ public static void AsDynamicLambda_StringEndsWith_ShouldWork()
724724
var filter = Filter.Function(
725725
Filter.Variable("Name"),
726726
"EndsWith",
727-
Filter.Constant("son"));
727+
new[] { Filter.Constant("son") });
728728

729729
// Act
730730
var lambda = filter.AsDynamicLambda<Person>();
@@ -1135,7 +1135,7 @@ public static void AsDynamicLambda_NestedObjectMethodCall_ShouldWork()
11351135
var filter = Filter.Function(
11361136
Filter.Variable("Address.City"),
11371137
"Contains",
1138-
Filter.Constant("York"));
1138+
new[] { Filter.Constant("York") });
11391139

11401140
// Act
11411141
var lambda = filter.AsDynamicLambda<Person>();
@@ -1262,7 +1262,7 @@ public static void AsDynamicLambda_ListContainsExtension_ShouldWork()
12621262
var filter = Filter.Function(
12631263
Filter.Variable("Tags"),
12641264
"Contains",
1265-
Filter.Constant("developer"));
1265+
new[] { Filter.Constant("developer") });
12661266

12671267
// Act
12681268
var lambda = filter.AsDynamicLambda<Person>();

0 commit comments

Comments
 (0)