CODEWITHSUNDEEP
ajaxjquery
zafrada
zafrada

Reputation: 61

Check if image exists, if not load it

I need to load an image if if hasn't been loaded yet. For this i'm using the error function like this:

$.ajax({  
type: "POST",  
url: "inc/functions.php",  
data: { productID: productIDVal, action: "addToCart"},  
success: function(theResponse) {
 busy=false;
 $('#buyButton')
  .error(function(){
   var t = $("<img id='buyButton' src='images/checkout.png' />");
   $.append(t);
 });
}  
});

But it is not working. Am i doing something wrong here?

Thanks in advance.

Upvotes: 1

Views: 1479

Answers (2)

bowsersenior
bowsersenior

Reputation: 12574

You need to specify where to append the image:

$('#buyButton').append(t);  // NOT $.append(t);
// OR $(this).append(t);

More info here

Some general tips on debugging javascript.

Quick and dirty: Put an alert message on the first line of the error function like alert('inside error'). Then load the page and see if the alert message shows up. You can put variables inside the alert message to see what their values are. If you don't see an alert message it means that the code is not even being loaded for some reason, so you have to put an alert message at an earlier point. (This can get very tedious).

Better way: Start using Firebug or Safari's Web Inspector to debug the javascript. Just put debugger anywhere in your code and when the browser gets to that line of code, it will stop and give you a console with access to all variables and functions available at that point in the code.

The problem may be with the AJAX request. See what it is returning by trying this code:

$.ajax({  
  type: "POST",  
  url: "inc/functions.php",  
  data: { productID: productIDVal, action: "addToCart"},
  success: function(data){ alert('success!'); },
  error: function(XMLHttpRequest, textStatus, errorThrown){ alert(errorThrown) ;}
})

You can replace the alert messages with debugger to do further inspection of what is going on with Firebug.

Upvotes: 3

kobe
kobe

Reputation: 15835

you should move the code to error block

remove from the success block 

error: function(request,error) {
  var t = $("<img id='buyButton' src='images/checkout.png' />");
   $(this).append(t);
  }

the skeleton goes like this

$.ajax({

 },
 beforeSend: function() {

 },

 error: function(request,error) {
  var t = $("<img id='buyButton' src='images/checkout.png' />");
   $(this).append(t);
  },
  success: function(request) {

  } // End success
}); // End ajax method

Upvotes: 0

Related Questions