Skip to content

Commit 56cf588

Browse files
author
Bart Koelman
committed
Changed DisableQueryStringAttribute.ParameterNames type from IReadOnlyCollection to IReadOnlySet
1 parent dfee19f commit 56cf588

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/JsonApiDotNetCore/Controllers/Annotations/DisableQueryStringAttribute.cs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,14 +22,15 @@ namespace JsonApiDotNetCore.Controllers.Annotations
2222
public sealed class DisableQueryStringAttribute : Attribute
2323
{
2424
public static readonly DisableQueryStringAttribute Empty = new(StandardQueryStringParameters.None);
25-
public IReadOnlyCollection<string> ParameterNames { get; }
25+
26+
public IReadOnlySet<string> ParameterNames { get; }
2627

2728
/// <summary>
2829
/// Disables one or more of the builtin query parameters for a controller.
2930
/// </summary>
3031
public DisableQueryStringAttribute(StandardQueryStringParameters parameters)
3132
{
32-
var parameterNames = new List<string>();
33+
var parameterNames = new HashSet<string>();
3334

3435
foreach (StandardQueryStringParameters value in Enum.GetValues(typeof(StandardQueryStringParameters)))
3536
{
@@ -50,7 +51,7 @@ public DisableQueryStringAttribute(string parameterNames)
5051
{
5152
ArgumentGuard.NotNullNorEmpty(parameterNames, nameof(parameterNames));
5253

53-
ParameterNames = parameterNames.Split(",").ToList();
54+
ParameterNames = parameterNames.Split(",").ToHashSet();
5455
}
5556

5657
public bool ContainsParameter(StandardQueryStringParameters parameter)

0 commit comments

Comments
 (0)