Ajax running only once

Question

I have an ajax function, and I expect it to run 1912 times, but it only runs once, for some reason. I'm using startAt, and stopAt to determine when it should stop running, but it's not working for some reason. What am I doing wrong?

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.3/jquery.min.js"></script>
<script type = "text/javascript"> 
function callAjax(gotoUrl, link, startAt, stopAt, output) {

  $.ajax({
    type: "POST",
    url: gotoUrl,
    data: { link : link },
    error: function(xhr,status,error){
      alert("error");
    },
    success:function(data) {
      document.getElementById(output).innerHTML += startAt;
    },
    complete:function(data) {
      startAt++;
      var link = data; 
      if (startAt < stopAt) {
        setTimeout(function(){ 
            callAjax(gotoUrl, link, startAt, stopAt, output) 
        }, 100);
      }
    }
  });

} //end of function callAjax()  
</script>

<body onload = 'callAjax("test1.php", "link", 1, 1912, "output")'>
<div id = "output"></div>

Result:

Expected Result:

you are putting the startAt++; inside an async call, why not put it befora actually performing the ajax call? — jstuartmilne
– jstuartmilne, Commented Oct 27, 2015 at 1:47
Why do you need the setTimeout() call? Just call the function... — Chris
– Chris, Commented Oct 27, 2015 at 1:50
@Sudakatux It works when I put it inside success. What that suppose to happen? — frosty
– frosty, Commented Oct 27, 2015 at 1:54
Try console.logging data inside the complete event to ensure this is even firing... — Chris
– Chris, Commented Oct 27, 2015 at 1:55

user764357 · Accepted Answer · 2015-10-27 02:00:02Z

2

The issue is on this line:

var link = data;

you are reassigning the value of link to be the returned data.

You then immediately call this inside the timeout:

callAjax(gotoUrl, link, startAt, stopAt, output)

But link isn't a link any more its an object, hence jquery errors out, and silently dies after one iteration.

Removing that line makes the code function fine, you just need to store the data in another variable and it'll work.

Here's a fiddle with the functional code with just that line commented out.

answered Oct 27, 2015 at 2:00

user764357

Sign up to request clarification or add additional context in comments.

4 Comments

frosty Over a year ago

Is there a way to convert the object back to string?

user764357 Over a year ago

It really depends on what that object is.

frosty Over a year ago

There are different types of object? What are the main types?

frosty Over a year ago

Wait. You mean like arrays, and such?

Collectives™ on Stack Overflow

Ajax running only once

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related