Fix for CheckExpressionForTypes (checking of MethodCallExpression )

Author: Oleg Nadymov

src/ServiceStack.OrmLite/Expressions/SqlExpression.cs

@@ -1525,9 +1525,12 @@ protected bool CheckExpressionForTypes(Expression e, ExpressionType[] types)
                 {
                     for (var i = 0; i < methodCallExpr.Arguments.Count; i++)
                     {
-                        if (CheckExpressionForTypes(methodCallExpr.Arguments[0], types))
+                        if (CheckExpressionForTypes(methodCallExpr.Arguments[i], types))
                             return true;
                     }
+
+                    if (CheckExpressionForTypes(methodCallExpr.Object, types))
+                        return true;
                 }
 
                 var unaryExpr = e as UnaryExpression;

tests/ServiceStack.OrmLite.Tests/ExpressionVisitorTests.cs

@@ -564,6 +564,17 @@ public void Can_Where_using_SqlIn_filter()
             Assert.That(target.Count, Is.EqualTo(1));
         }
 
+        [Test]
+        public void Can_Where_using_IfConcat_filter()
+        {
+            System.Linq.Expressions.Expression<Func<TestType, bool>> filter = x => (String.Concat("Text: ", x.TextCol) == null ? null : String.Concat("Text: ", x.TextCol)).EndsWith("asdf");
+            var q = Db.From<TestType>().Where(filter);
+            Assert.That(q.ToSelectStatement(), Does.Contain("Text"));
+
+            var target = Db.Select(q);
+            Assert.That(target.Count, Is.EqualTo(1));
+        }
+
         private int MethodReturningInt(int val)
         {
             return val;