Skip to content

Commit 328c380

Browse files
committed
Fix the numberstyles
1 parent 2553508 commit 328c380

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

src/Microsoft.OpenApi.Readers/ParseNodes/ValueNode.cs

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -52,23 +52,21 @@ public override IOpenApiAny CreateAny()
5252
return new OpenApiBoolean(false);
5353
}
5454

55-
if (int.TryParse(value, NumberStyles.None, CultureInfo.InvariantCulture, out var intValue))
55+
// The NumberStyles below are the default ones based on
56+
// https://docs.microsoft.com/en-us/dotnet/api/?view=netframework-4.7.2
57+
if (int.TryParse(value, NumberStyles.Integer, CultureInfo.InvariantCulture, out var intValue))
5658
{
5759
return new OpenApiInteger(intValue);
5860
}
5961

60-
if (long.TryParse(value, NumberStyles.None, CultureInfo.InvariantCulture, out var longValue))
62+
if (long.TryParse(value, NumberStyles.Integer, CultureInfo.InvariantCulture, out var longValue))
6163
{
62-
return
63-
new OpenApiLong(
64-
longValue);
64+
return new OpenApiLong(longValue);
6565
}
6666

67-
if (double.TryParse(value, NumberStyles.None, CultureInfo.InvariantCulture, out var doubleValue))
67+
if (double.TryParse(value, NumberStyles.Float | NumberStyles.AllowThousands, CultureInfo.InvariantCulture, out var doubleValue))
6868
{
69-
return
70-
new OpenApiDouble(
71-
doubleValue); // Note(darrmi): This may be better as decimal. Further investigation required.
69+
return new OpenApiDouble(doubleValue);
7270
}
7371

7472
if (DateTimeOffset.TryParse(value, CultureInfo.InvariantCulture, DateTimeStyles.None, out var dateTimeValue))

0 commit comments

Comments
 (0)