Skip to content

Commit 9546bcb

Browse files
committed
Use char overload instead of passing a single-char string
1 parent 215df74 commit 9546bcb

File tree

5 files changed

+5
-5
lines changed

5 files changed

+5
-5
lines changed

src/JsonApiDotNetCore.Annotations/TypeExtensions.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public static string GetFriendlyTypeName(this Type type)
4848
if (type.IsGenericType)
4949
{
5050
string typeArguments = type.GetGenericArguments().Select(GetFriendlyTypeName).Aggregate((firstType, secondType) => $"{firstType}, {secondType}");
51-
return $"{type.Name[..type.Name.IndexOf("`", StringComparison.Ordinal)]}<{typeArguments}>";
51+
return $"{type.Name[..type.Name.IndexOf('`')]}<{typeArguments}>";
5252
}
5353

5454
return type.Name;

src/JsonApiDotNetCore/QueryStrings/FilterQueryStringParameterReader.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public virtual bool CanRead(string parameterName)
5252
{
5353
ArgumentGuard.NotNullNorEmpty(parameterName);
5454

55-
bool isNested = parameterName.StartsWith("filter[", StringComparison.Ordinal) && parameterName.EndsWith("]", StringComparison.Ordinal);
55+
bool isNested = parameterName.StartsWith("filter[", StringComparison.Ordinal) && parameterName.EndsWith(']');
5656
return parameterName == "filter" || isNested;
5757
}
5858

src/JsonApiDotNetCore/QueryStrings/SortQueryStringParameterReader.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ public virtual bool CanRead(string parameterName)
4545
{
4646
ArgumentGuard.NotNullNorEmpty(parameterName);
4747

48-
bool isNested = parameterName.StartsWith("sort[", StringComparison.Ordinal) && parameterName.EndsWith("]", StringComparison.Ordinal);
48+
bool isNested = parameterName.StartsWith("sort[", StringComparison.Ordinal) && parameterName.EndsWith(']');
4949
return parameterName == "sort" || isNested;
5050
}
5151

src/JsonApiDotNetCore/QueryStrings/SparseFieldSetQueryStringParameterReader.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ public virtual bool CanRead(string parameterName)
5050
{
5151
ArgumentGuard.NotNullNorEmpty(parameterName);
5252

53-
return parameterName.StartsWith("fields[", StringComparison.Ordinal) && parameterName.EndsWith("]", StringComparison.Ordinal);
53+
return parameterName.StartsWith("fields[", StringComparison.Ordinal) && parameterName.EndsWith(']');
5454
}
5555

5656
/// <inheritdoc />

test/JsonApiDotNetCoreTests/IntegrationTests/IdObfuscation/HexadecimalCodec.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ public int Decode(string? value)
1515
return 0;
1616
}
1717

18-
if (!value.StartsWith("x", StringComparison.Ordinal))
18+
if (!value.StartsWith('x'))
1919
{
2020
throw new JsonApiException(new ErrorObject(HttpStatusCode.BadRequest)
2121
{

0 commit comments

Comments
 (0)