jQuery callback function needs to replace parameter using .each

Question

0.00/5 (No votes)

See more:

I have this function I want to use that will cycle through all the DIVs using .each in a page that have a certain class, then the DIV get replaced by the results of a .get. Here is the code:

PHP

$(function(){
    window.setInterval(function() {
    $(".cstat").each(function (i) {
        $.get("getcustomerstatus.asp?customerid="+this.id).done(function(data){$("#"+this.id).html(data);});
        })
    }, 10000);
})

The problem is that in the .done the "this.id" I believe gets evaluated at the return of the data from the .get. My goal is to replace all the DIV. So my overall question is there a way to make the this.id in the .done section get "inserted" at the time of the .get instead of the .done?

Thanks

Posted 28-May-13 6:03am

onemorecoke

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) · Accepted Answer · 2013-05-28T06:10:00

Solution 1

You can store that id of div before calling that .get() function like below.

JavaScript

$(function(){
    window.setInterval(function() {
    $(".cstat").each(function (i) {
        var currentDivId = this.id;

        $.get("getcustomerstatus.asp?customerid="+this.id).done(function(data){$("#" + currentDivId ).html(data);});
        })
    }, 10000);
})

Let me know it solves your problem or not.

Posted 28-May-13 6:10am

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ)

Comments

onemorecoke 28-May-13 13:53pm

Worked perfect! I am curious about why this fixed it. I could see the usefulness of both ways but wondered what triggered the "current" replacement vs the "later" replacement. Anyway, maybe for another day...thank you again

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 28-May-13 14:03pm

Most Welcome buddy... :)

The keyword "this" always refers to the current object/element.
As you were calling it inside that "done()", so it will refer that object not the div.
So, I stored it in a variable to use that inside the function.