How to use variable in JQuery select

Question

How do I modify the jquery to use the var instead of the '12'?

    var disableId = 12;

    if (e.currentTarget.checked) {

        $("input:checkbox[value=12]").attr("disabled", true);
    }
    else {
        $("input:checkbox[value=12]").attr("disabled", false);
    }

$("input:checkbox[value='"+disableId +"']).attr("disabled", true); — Rohit Tiwari
– Rohit Tiwari, Commented Jan 31, 2014 at 14:14
In case you go straight to Anton's upvoted answer, I'll just point out here what I've said in my answer: the if statement you're using is unnecessary as you can simply pass in e.currentTarget.checked in place of true or false. You should also use prop() instead of attr(). — James Donnelly
– James Donnelly, Commented Jan 31, 2014 at 14:25

James Donnelly · Accepted Answer · 2014-01-31 14:20:58Z

1

You should use jQuery's prop() method instead of attr() for boolean values; also there's no need for the IF statements at all as you can just set the disabled state to equal e.currentTarget.checked (which is either true or false):

var disableId = 12,
    state = e.currentTarget.checked;

$("input:checkbox[value='" + disableId + "']").prop("disabled", state);

answered Jan 31, 2014 at 14:20

James Donnelly

129k35 gold badges215 silver badges224 bronze badges

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

Comments

Anton · Accepted Answer · 2014-01-31 14:21:14Z

1

Open the string and add it

$("input:checkbox[value='"+disableId +"']").prop("disabled", true);

also you should use prop() instead of attr for boleean values

edited Jan 31, 2014 at 14:21

answered Jan 31, 2014 at 14:15

Anton

32.6k5 gold badges47 silver badges54 bronze badges

Collectives™ on Stack Overflow

How to use variable in JQuery select

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related