Problem with JSON

[Ivan Gazzola]

After last upgrade i've this problem with response.json

-----Controller  cut -------

if form.process().accepted:

     return response.json({'formkey':form.formkey,'html':XML(html_from_helpers)})

-----cut -------

The json generated by controller fails in this js because don't escape double quotes in html_from_helpers string. Is it a bug?  Before last update it worked fine ....

$('#form').submit(function(e){
e.preventDefault();
    $.ajax({
           type: "POST",
           url: "/my/url",
           data: $('#form').serialize(), // serializes the form's elements.
           success: function(json)
           {
               risposta=JSON.parse(json);; // show response from the php script.
                  $('#form:input[name="_formkey"]').val(risposta.formkey);
                  $('#myUL').html(risposta.html).listview('refresh');
                  $(".risposta_flash").remove();$('<div class="ui-body ui-body-b risposta_flash" style="margin:20px;">Stato di cura inserito</div>').prependTo("#content_principale").hide().slideDown();
           }
         });
return false;
});    

Thx

[Niphlod]

uhm. what does your shell print out of this ?

response.json({'html' : XML('<span class="foo">bar</span>')})

mine does

u'{"html": "<span class=\\"foo\\">bar</span>"}'

so it's not the issue you're describing.

[Ivan Gazzola]

Ok, solved :)

The issue was in some changes in Jquery.

Now JQuery parses ajax response before than it is passed to success function, so js fails if I call "parse" on it:

risposta=JSON.parse(json);
                      ^^^^^^^^^^^^^^^

THX

Ivan

[Niphlod]

to be fair, I think you encountered a minor glitch........ response.json() now sets the content-type to application/json (something that your app should have done already, but probably overlooked).
Returning a json string with text/html as content-type made jquery not aware that that string was a json, but only a string in the need to be parsed.
Moreover, JSON(something) is NOT a jquery function, it's a browser one, and not available on all browsers. That piece of code would never have worked on e.g. IE7.
Now the response is instead a json string "signalled" to be a json object, and jquery parses it correctly

<tl/dr> for the future...Don't EVER overlook content-type. Read about "dataType" on jquery's ajax().