CODEWITHSUNDEEP
jqueryajaxeventscallbackruby-on-rails-3
ncavig
ncavig

Reputation: 381

JQuery and Rails 3 ajax:failure callback

I'm using Rails 3 and JQuery 1.4.2 and am trying to bind to the ajax:failure callback on a given remote form submission. The callback works fine, however, the xhr variable that's passed back seems to lose the responseText attribute somehow.

Here's what my code looks like:

_form.html.haml

= form_for(object, :remote => true) do |f|
  = form fields and such...

Javascript somewhere...

 $('form').livequery('ajax:loading', function() {
    // what to do on ajax loading
}).livequery('ajax:success', function(data, status, xhr) {

}).livequery('ajax:failure', function(xhr, status, error) {
    alert(xhr.responseText);
});

I'm basically rendering the object's error messages from the controller so that I can display error notifications on this callback. The weird thing is I go into rails.js, lines 49-51 

error: function (xhr, status, error) {
  el.trigger('ajax:failure', [xhr, status, error]);
}

and manually write to the console responseText, it works the way I would expect.

Am I doing something wrong? How would the xhr object change from the rails.js to my bind?

Upvotes: 3

Views: 3472

Answers (1)

ncavig
ncavig

Reputation: 381

I figured out the answer but am confused to why it is so. So rails.js triggers an ajax:failure event by using the following code snippet

$.ajax({
    url: url,
    data: data,
    dataType: dataType,
    type: method.toUpperCase(),
    ...
    error: function (xhr, status, error) {
        el.trigger('ajax:failure', [xhr, status, error]);
    }
});

When I bind to that event, 

$('form').livequery('ajax:failure', function(xhr, status, error) {

the status var appears to have the responseText attribute and not the xhr as I would have thought.

So, 

console.log(status.responseText)

spits out my response text.

The correct bind should look like this

('form').livequery('ajax:failure', function(event, xhr, status, error) {

Since the first variable is the event that fired.

Upvotes: 6

Related Questions