get multiple checked value in javascript/jquery

Question

I have status id defined in my div tag which is present in a phtml file .Now, I need to fetch its value into the seprate javascript file.

I have use jquery push method for this.

Code in phtml file :

<table class="abc">
<tbody id="checkboxes">
<tr class="r<?php echo $item['id']; ?><?php echo ($item['status_low']?' status-low':''); ?>">
                    <td><input type="checkbox" name="item" class="row" statusid="<?php echo $item['status_id']; ?>"</td>
                    <td><?php echo $item['id']; ?></td>
                    <td><?php echo $item['title']; ?></td>
</tr>
</tbody>
</table>

Code in Javascript File :

 var selected = [];
    $('#checkboxes input:checked').each(function() {
       if ($(this).checked) {
           selected.push($(this).attr('statusid'));
       }
    });
    console.log(selected);

When I print selected array, I get a blank output .

Can anyone tell me where am I going wrong ?

can you post the rendered HTML, I don't see a container `#checkboxes` there. — Mritunjay, Aug 28 '14 at 06:23
If you have already chosen `checked` `input`s, then what for you checked it again using `if`, also doing it in incorrect way? — Regent, Aug 28 '14 at 06:26
@regent : i have used that code from http://stackoverflow.com/questions/2155622/get-a-list-of-checked-checkboxes-in-a-div-using-jquery — mmt, Aug 28 '14 at 06:27
There is no `if` in mentioned question. Just remove `if` in your code, and it will work. — Regent, Aug 28 '14 at 06:28

Kartikeya Khosla · Answer 1 · 2014-08-28T06:33:30.690

1

Try this :-

var selected = [];
$("input[type='checkbox']").each(function(){
  if($(this).is(':checked'))
  {
    selected.push($(this).attr('statusid'));
  }
});

OR

var selected = [];
$("#checkboxes input[type='checkbox']").each(function(){
  if($(this).is(':checked'))
  {
    selected.push($(this).attr('statusid'));
  }
});

OR

var selected = [];
$("#checkboxes input[type='checkbox']:checked").each(function(){
    selected.push($(this).attr('statusid'));
});

edited Aug 28 '14 at 06:33

answered Aug 28 '14 at 06:21

Kartikeya Khosla

18,743
8
43
69

Even this works out as required ! Thanks exception for exceptional help :) – mmt Aug 28 '14 at 06:47

score 1 · Accepted Answer · answered Aug 28 '14 at 06:31

1

Just remove your if condition like bellow. Your selector is just for selected checkboxes, So you don't need that if anyway.

$('#checkboxes input:checked').each(function() {
       selected.push($(this).attr('statusid'));
});

DEMO

answered Aug 28 '14 at 06:31

Mritunjay

25,338
7
55
68

That's I'm talking about :) – Regent Aug 28 '14 at 06:32

score 0 · Answer 3 · answered Aug 28 '14 at 06:29

0

Change your JQuery code like this,

$('#checkboxes input:checked')

to

$("#checkboxes input[type='checked']")

your code is:::

var selected = [];
    $("#checkboxes input[type='checked']").each(function() {

           selected.push($(this).attr('statusid'));

    });
    console.log(selected);

answered Aug 28 '14 at 06:29

chandu

2,276
3
20
35

The problem is in incorrect and unnecessary `$(this).checked`, not in inputs' selector. – Regent Aug 28 '14 at 06:30

score 0 · Answer 4 · answered Aug 28 '14 at 06:31

"#checkboxes input:checked" selector already selecting checked checkboxes so you don't need to check if checkbox is checked or not. See here http://api.jquery.com/checked-selector/

var selected = $.map('#checkboxes input:checked',function(ck,i){
   return $(ck).attr('statusid');
})

score 0 · Answer 5 · answered Aug 28 '14 at 06:40

0

You could try this one

 var selected = $('#checkboxes input:checkbox:checked.row').map(function () {
             var cbId = this.id;
             return cbId;
         }).get();

answered Aug 28 '14 at 06:40

xei2k

536
4
9

get multiple checked value in javascript/jquery

5 Answers5