CSHARP-1650: Combine if statements.

rstam · rstam · commit 0c8e3b3482a6 · 2016-09-24T18:26:34.000-04:00
diff --git a/src/MongoDB.Driver/Linq/Translators/AggregateLanguageTranslator.cs b/src/MongoDB.Driver/Linq/Translators/AggregateLanguageTranslator.cs
@@ -977,7 +977,7 @@ private bool TryTranslateStringMethodCall(MethodCallExpression node, out BsonVal
                     }
                     break;
                 case "Split":
-                    if (node.Arguments.Count < 1)
+                    if (node.Arguments.Count < 1 || node.Arguments.Count > 2)
                     {
                         return false;
                     }
@@ -1008,10 +1008,6 @@ private bool TryTranslateStringMethodCall(MethodCallExpression node, out BsonVal
                             return false;
                         }
                     }
-                    if (node.Arguments.Count > 2)
-                    {
-                        return false;
-                    }
                     result = new BsonDocument("$split", new BsonArray
                     {
                         field,

Original file line number	Diff line number	Diff line change
`@@ -977,7 +977,7 @@ private bool TryTranslateStringMethodCall(MethodCallExpression node, out BsonVal`
`977`	`977`	`}`
`978`	`978`	`break;`
`979`	`979`	`case "Split":`
`980`		`- if (node.Arguments.Count < 1)`
	`980`	`+ if (node.Arguments.Count < 1 \|\| node.Arguments.Count > 2)`
`981`	`981`	`{`
`982`	`982`	`return false;`
`983`	`983`	`}`
`@@ -1008,10 +1008,6 @@ private bool TryTranslateStringMethodCall(MethodCallExpression node, out BsonVal`
`1008`	`1008`	`return false;`
`1009`	`1009`	`}`
`1010`	`1010`	`}`
`1011`		`- if (node.Arguments.Count > 2)`
`1012`		`- {`
`1013`		`- return false;`
`1014`		`- }`
`1015`	`1011`	`result = new BsonDocument("$split", new BsonArray`
`1016`	`1012`	`{`
`1017`	`1013`	`field,`