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
– Mritunjay, Commented Aug 28, 2014 at 6:23
If you have already chosen checked inputs, then what for you checked it again using if, also doing it in incorrect way? — Regent
– Regent, Commented Aug 28, 2014 at 6:26
@regent : i have used that code from stackoverflow.com/questions/2155622/… — mmt
– mmt, Commented Aug 28, 2014 at 6:27

Mritunjay · Accepted Answer · 2014-08-28 06:31:07Z

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, 2014 at 6:31

Mritunjay

25.9k7 gold badges57 silver badges71 bronze badges

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

1 Comment

Regent Over a year ago

That's I'm talking about :)

Kartikeya Khosla · Accepted Answer · 2014-08-28 06:33:30Z

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, 2014 at 6:33

answered Aug 28, 2014 at 6:21

Kartikeya Khosla

18.9k8 gold badges46 silver badges70 bronze badges

1 Comment

mmt Over a year ago

Even this works out as required ! Thanks exception for exceptional help :)

chandu · Accepted Answer · 2014-08-28 06:29:14Z

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, 2014 at 6:29

chandu

2,2763 gold badges23 silver badges35 bronze badges

1 Comment

Regent Over a year ago

The problem is in incorrect and unnecessary $(this).checked, not in inputs' selector.

A.T. · Accepted Answer · 2014-08-28 06:31:09Z

0

"#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');
})

answered Aug 28, 2014 at 6:31

A.T.

26.7k8 gold badges50 silver badges68 bronze badges

Comments

xei2k · Accepted Answer · 2014-08-28 06:40:58Z

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, 2014 at 6:40

xei2k

5564 silver badges9 bronze badges

Collectives™ on Stack Overflow

get multiple checked value in javascript/jquery

5 Answers 5

1 Comment

1 Comment

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

1 Comment

1 Comment

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related