Jquery checkbox.checked always returns false

Question

$("tbody input[name='rooms_to_book']").on('change',function(){
    var current_row = $(this).closest("tr");
    var current_checkbox = current_row.find("input[name=checkbox]");
    if(current_checkbox.checked) {
        var price_per_night = parseFloat(current_row.find("td[name=price]").text().replace('$',''));
        var rooms_to_book = parseInt(current_row.find("input[name=rooms_to_book]").val());
        rooms_to_book = rooms_to_book ? rooms_to_book : 0;
        total_price = price_per_night * rooms_to_book;
    }else{
        total_price = 0;
    }
    current_row.find("td[name=total_price]").text('$'+ total_price);
});

The current_checkbox.checked always returns false, the html is correct.

The HTML may be correct but you need to add it to your question. — j08691, Jan 14 '14 at 17:55
possible duplicate of [JS .checked vs jquery attr('checked'), what is the difference?](http://stackoverflow.com/questions/15081335/js-checked-vs-jquery-attrchecked-what-is-the-difference) — Felix Kling, Jan 14 '14 at 17:59

score 11 · Accepted Answer · answered Jan 14 '14 at 17:56

11

jQuery objects do not have a checked property. Replace

current_checkbox.checked

with

current_checkbox.prop('checked')

http://api.jquery.com/prop

answered Jan 14 '14 at 17:56

Blazemonger

86,267
25
136
177

score 5 · Answer 2 · answered Jul 05 '14 at 05:51

5

You are using current_checkbox.checked, checkbox has not any 'checked' properties, but you can use $('#id').is(':checked')

answered Jul 05 '14 at 05:51

Kerem Kulak

51
1
2

score 2 · Answer 3 · edited Jan 14 '14 at 18:11

2

You're using a jQuery element, not a DOM element. Use:

if(current_checkbox.prop("checked")) {

} //just for Steve <3

edited Jan 14 '14 at 18:11

Steve

8,469
6
39
53

answered Jan 14 '14 at 17:57

Sterling Archer

21,348
17
79
113

score 1 · Answer 4 · answered Jan 14 '14 at 17:57

1

current_row.find("input[name=checkbox]")

I'm fairly sure this returns a jQuery object, not the underlying checkbox element.

if(current_checkbox[0].checked)

Should work.

answered Jan 14 '14 at 17:57

Niet the Dark Absol

311,322
76
447
566

Jquery checkbox.checked always returns false

4 Answers4