Parsing an integer value when JSON contains a decimal point about fleece HOT 9 OPEN

Seems like F# Data us fixing this as well: https://www.nuget.org/packages/FSharp.Data/3.0.0-beta4 :

(Breaking Change) Don't silently convert decimals and floats to integers in JsonProvider.

from fleece.

This also causes trouble when trying to do lensing.

from fleece.

JNumber s implies that s is JsonPrimitive ...

from fleece.

fsprojects/FSharp.Data#1185

from fleece.

I think this behavior is not correct at all.
Anyway, I also think this should be corrected in the original library (as FSharp.Data just did).
We should forward this issue to Newtonsoft and System.Json
Also we should note that Fleece is a Json mapper for different libraries, since different libraries have different behavior (there is no agreed standard for Json <-> .NET) it's not up to Fleece to try to fix them.

from fleece.

I agree. It only makes sense to align with the libraries. Right now JNumber active pattern has some interesting behavior.

from fleece.

I don't know if this is related:

1 |> toJson |> string;;
val it : string = "1"

but

(2,1) |> toJson |> string;;
val it : string = "[
  2.0,
  1.0
]"

from fleece.

We also see this kind of behavior: JamesNK/Newtonsoft.Json#1400
So I guess it's a tradeoff what Json parser you use.

from fleece.

I've added a PR that documents the behavior. Note that Fsharp.Data works the way you want.

from fleece.