On second thoughts John, i have to agree with you, especially as it
can be implemented using a filter...
function myfilter( type, data ) {
return type === 'json' ? JSON.parse(data, reviver) : data;
}
at least this would error if JSON didn't exist, and not silently
produce different results.
Regards
- Mark
2009/8/18 John Resig <[email protected]>:
> "that old piece of missing functionality"? Being what? Wholesale importing
> json2.js? That doesn't seem practical - especially since if json2.js is
> already included in a page (or if the native JSON object exists) we use it.
>
> --John
>
>
> On Mon, Aug 17, 2009 at 12:14 PM, Daniel Friesen <[email protected]>
> wrote:
>>
>> IIRC json2 supports it... We could get around to fixing that old piece
>> of missing functionality in jQuery and kill two birds with one stone.
>>
>> ~Daniel Friesen (Dantman, Nadir-Seen-Fire) [http://daniel.friesen.name]
>>
>> John Resig wrote:
>> > I'd be wary of adding that - especially since it wouldn't work for
>> > browsers that don't support JSON.parse.
>> >
>> > --John
>> >
>> >
>> > On Mon, Aug 17, 2009 at 3:12 AM, Mark Gibson <[email protected]
>> > <mailto:[email protected]>> wrote:
>> >
>> >
>> > Hi, any chance we could add a reviver option to the ajax settings
>> > for
>> > JSON requests?
>> > This could also be used by other plugins that parse JSON data too.
>> >
>> > - Mark
>> >
>> > Index: src/ajax.js
>> > ===================================================================
>> > --- src/ajax.js (revision 6529)
>> > +++ src/ajax.js (working copy)
>> > @@ -182,7 +182,8 @@
>> > json: "application/json, text/javascript",
>> > text: "text/plain",
>> > _default: "*/*"
>> > - }
>> > + },
>> > + jsonReviver: null
>> > },
>> >
>> > // Last-Modified header cache for next request
>> > @@ -565,7 +566,7 @@
>> > // Get the JavaScript object, if JSON is
>> > used.
>> > if ( type === "json" ) {
>> > if ( typeof JSON === "object" &&
>> > JSON.parse ) {
>> > - data = JSON.parse( data );
>> > + data = JSON.parse( data,
>> > s.jsonReviver );
>> > } else {
>> > data = (new
>> > Function("return " + data))();
>> > }
>> >
>> >
>> >
>> >
>> > >
>>
>>
>
>
> >
>
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"jQuery Development" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/jquery-dev?hl=en
-~----------~----~----~----~------~----~------~--~---
