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 <jere...@gmail.com>: > "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 <nadir.seen.f...@gmail.com> > 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 <jollyt...@gmail.com >> > <mailto:jollyt...@gmail.com>> 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 jquery-dev@googlegroups.com To unsubscribe from this group, send email to jquery-dev+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/jquery-dev?hl=en -~----------~----~----~----~------~----~------~--~---