Update code to use ternary operator

gilsonjoanelo · gilsonjoanelo · commit b2c7eb9de569 · 2024-09-25T20:09:33.000-03:00
diff --git a/src/FirebirdSql.EntityFrameworkCore.Firebird/Storage/Internal/FbSqlGenerationHelper.cs b/src/FirebirdSql.EntityFrameworkCore.Firebird/Storage/Internal/FbSqlGenerationHelper.cs
@@ -30,12 +30,8 @@ public FbSqlGenerationHelper(RelationalSqlGenerationHelperDependencies dependenc
 	public virtual string StringLiteralQueryType(string s, bool isUnicode = true)
 	{
 		var length = MinimumStringQueryTypeLength(s);
-		EnsureStringLiteralQueryTypeLength(length);
-		if(isUnicode)
-		{
-			return $"VARCHAR({length}) CHARACTER SET UTF8";
-		}
-		return $"VARCHAR({length})";
+		var charset = isUnicode ? " CHARACTER SET UTF8" : "";
+		return $"VARCHAR({length}){charset}";
 	}
 
 	public virtual string StringParameterQueryType(bool isUnicode)

Original file line number	Diff line number	Diff line change
`@@ -30,12 +30,8 @@ public FbSqlGenerationHelper(RelationalSqlGenerationHelperDependencies dependenc`
`30`	`30`	`public virtual string StringLiteralQueryType(string s, bool isUnicode = true)`
`31`	`31`	`{`
`32`	`32`	`var length = MinimumStringQueryTypeLength(s);`
`33`		`- EnsureStringLiteralQueryTypeLength(length);`
`34`		`- if(isUnicode)`
`35`		`- {`
`36`		`- return $"VARCHAR({length}) CHARACTER SET UTF8";`
`37`		`- }`
`38`		`- return $"VARCHAR({length})";`
	`33`	`+ var charset = isUnicode ? " CHARACTER SET UTF8" : "";`
	`34`	`+ return $"VARCHAR({length}){charset}";`
`39`	`35`	`}`
`40`	`36`
`41`	`37`	`public virtual string StringParameterQueryType(bool isUnicode)`