Re: Re: Removing basic types from our JSON parser

Lukas, Reading the spec it clearly says in the pre-amble that JSON can represent both primitive and complex data types From RFC: "JSON can represent four primitive types (strings, numbers, booleans, and null) and two structured types (objects and arrays)." The RFC also says that when it comes to parsers anything that matches the JSON grammar is valid, it even goes further that accepting "extensions" on top of the grammar is fine as well RFC: "A JSON parser MUST accept all texts that conform to the JSON grammar. A JSON parser MAY accept non-JSON forms or extensions." On 13-Dec-08, at 1:05 PM, Lukas Kahwe Smith wrote:

On 13.12.2008, at 18:59, Ilia Alshanetsky wrote:

On 12-Dec-08, at 10:50 PM, Scott MacVicar wrote:

Hi All, Basic types were added to our JSON decoder to PHP 5.2.1, this allows one to use json_encode / json_decode on any of our scalar types. Omar correctly identified #38680 as not a bug but it appears that Ilia added support for this anyway violating the RFC [1]. Maybe there was a reason for this but I'm not sure why?

PHP is typeless language, IMO its conductive to this design to allow encoding of basic types via json_encode, it reduces the code when communicating to/form JavaScript to avoid type detection. While it does violate the RFC IMO the convenience of the feature is definitely worth it.

well seems to me like its safer to stick with RFC as the default. if at all this should be optional. JSON is a standard and this requires everybody to follow it. so using something else should be a conscious decision that is also visible during code review. regards, Lukas Kahwe Smith mls@pooteeweet.org

Ilia Alshanetsky