search

VBA-tools / VBA-JSON

JSON conversion and parsing for VBA
MIT License
1.79k stars 573 forks source link

Error with config JsonOptions.UseDoubleForLargeNumbers #208

Open victortmagalhaes opened 3 years ago

victortmagalhaes commented 3 years ago

In line 238, there is a negation of the option "UseDoubleForLargeNumbers", when the correct shoulb be just the use of the option.

Instead of If Not JsonOptions.UseDoubleForLargeNumbers And json_StringIsLargeNumber(JsonValue) Then should be If JsonOptions.UseDoubleForLargeNumbers And json_StringIsLargeNumber(JsonValue) Then

The default for this option is false, but when there is a "not" before, it cause the VBA JSON parse to transform a big String that seams like a number, in a number, when we really want a string.