Re: [Json] The names within an object SHOULD be unique.

[Allen Wirfs-Brock <[e-mail]>]

Just got signed on to this list...
(and this message didn't seem to make it there the first time, so sorry if this is duplicated)

There is also a discussion of this topic going on at [e-mail] which I'm trying to redirect here.

My recommendation from the es-disuss thread is below.

Allen Wirfs-Brock
ECMAScript Language Spec. Project Editor


On Jun 6, 2013, at 8:24 AM, Paul Hoffman wrote:

> On Jun 5, 2013, at 4:26 PM, Douglas Crockford <[e-mail]> wrote:
> 
>> If duplicate names are encountered, the parse MAY fail (because some implementations correctly do that), or it MAY succeed by accepting the last duplicated key:value pair.
> 
> The new text should use language that is already in RFC 4627; "key" is not such a word. To be fair to implementers, the new document also needs to deal with both emitters and parsers.
> 
> Proposal:
> 
> In Section 2.2:
> Current:
>   The names within an object SHOULD be unique.
> Proposed:
>   If the names within an object are not unique, the result of parsing the 
>   object is unpredictable, and the parse may even fail completely. Thus,
>   the names within an object SHOULD be unique.
> 
> In Section 4, add a new paragraph:
>   If a parser encounters an object with duplicate names, the parser MAY
>   fail to parse the JSON text; if the parser accepts objects with duplicate
>   names, it SHOULD accept only the last name/value pair that has the
>   duplicate name. 
> 
> --Paul Hoffman

Some that should be pointed out is that one reason JSON needs to accept duplicate field names is that some people currently use them to add  comments to JSON datasets:

{
"//": "This is a comment about my data",
"//":  "that takes more than one line",
"field1" : 1,
"field2" : 2
}

The above is valid according to the existing RFC and is accepted as valid JSON by the existing ECMAScript  JSON.parse spec. We don't want to invalid existing JSON datasets that use this technique so duplicate keys MUST be accepted.

What I would say, as a replacement for the current text  <section 2.2> is:

         The names within an object SHOULD be unique.  If a key is duplicated, a parser MUST take  <<use?? interpret??>> only the last of the duplicated key pairs.

I would be willing to loose the first sentence (containing the SHOULD) entirely as it doesn't add any real normative value.

Given that duplicate names have historically been valid, that some people use them, and that the standard ECMAScript JSON parser accepts them I don't see why allowing a parser to reject duplicate names is helpful.