Hi John, done - see http://dev.jquery.com/ticket/3565
Hope that's useful... roBman On Sat, 2008-11-01 at 13:28 -0400, John Resig wrote: > This looks like an interesting patch - could you add it to the bug tracker? > http://dev.jquery.com/ > > Thanks. > > --John > > > > On Fri, Oct 31, 2008 at 9:22 PM, Rob Manson > <[EMAIL PROTECTED]> wrote: > > > > Hi, > > > > not sure if you're happy for patches to be posted directly here...so > > feel free to tell me where to patch it 8) > > > > > > roBman > > > > > > > > BUG: > > httpData: function in ajax.js (used by getJSON() calls or get() calls > > where type == "json") doesn't currently test the eval is successful. > > > > e.g. it just assumes that valid JSON was returned and eval will succeed, > > silently failing if this is the case. > > > > FIX: > > Here's a patch to bring it in line with the other functions above. > > > > e.g. httpSuccess and httpNotModified > > > > NOTE: Let me know if there's a more "preferred" way to handle this error > > > > DIFF against 1.2.6 from svn: > > Index: src/ajax.js > > =================================================================== > > --- src/ajax.js (revision 5918) > > +++ src/ajax.js (working copy) > > @@ -478,9 +478,12 @@ > > > > // Get the JavaScript object, if JSON is used. > > if ( type == "json" ) > > - data = eval("(" + data + ")"); > > - > > - return data; > > + // Check to make sure the JSON returned evaluates > > correctly > > + try { > > + data = eval("(" + data + ")"); > > + return data; > > + } catch(e) {} > > + return false; > > }, > > > > // Serialize an array of form elements or a set of > > > > > > > > > > > > > > > --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---
