selecting multiple checkbox in jquery

Question

I wrote a jquery code to send values to a php file

$('#def_formSubmit').live("click",function(){

        var query_string = '';
                    
                    $("input[@type='checkbox'][@name='assotop']").each(
                        function()
                        {
                            if(this.checked)
                            {
                                query_string += "&assotop[]=" + this.value;
                            }
                        });

    
    var def_tags = $("#tags").val();
    
    var dataString = 'def_tags='+ def_tags + query_string  ;

    $.ajax({
        type: 'POST',
        url: 'post.php',
        data: dataString,
        cache: false,
        beforeSend: function() {
            $("#QRresult").html("<img src='images/loading.gif' />");
        },
        success: function(data5) {
            $("#QRresult").html(data5);
        }
    });
    return false;
});

But it does not work and does not sendi checkbox values to my php file.

I think this way of getting checkbox values is a old approach and not working for jquery 1.4.

user113716 · Accepted Answer · 2010-11-07 18:30:54Z

3

Remove the @ from your selector. This changed as of 1.3

$("input[@type='checkbox'][@name='assotop']").each(

should be:

$("input[type='checkbox'][name='assotop']").each(

~~You could also do:~~

~~$("input:checkbox[name='assotop']").each(~~

EDIT: Added input to the selector as noted by @lonesomeday.

EDIT: I guess I'd use the one with input[type='checkbox'] so that querySelectorAll will be used in browsers that support it.

edited Nov 7, 2010 at 18:30

answered Nov 7, 2010 at 17:59

user113716

323k64 gold badges454 silver badges441 bronze badges

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

8 Comments

Russ Cam Over a year ago

perhaps use input:checkbox too?

lonesomeday Over a year ago

+1, but input:checkbox will be much quicker, since it doesn't have to select every element and then check it for being an input element.

user113716 Over a year ago

@lonesomeday - I thought :checkbox took care of that. Perhaps I'm mistaken. I'll update. EDIT: Looking at the docs, I see you're correct. Thanks. :o)

Sarfraz Over a year ago

@lonesomeday: Why not just use :checkbox which is much faster, you don't need input there with it.

user113716 Over a year ago

@Sarfraz - From the docs: $(':checkbox') is equivalent to $('*:checkbox'), so $('input:checkbox') should be used instead.

|

Srdjan Dejanovic · Accepted Answer · 2017-09-14 12:58:09Z

1

Try this code:

$(document).ready(function(){
    $("#selectAll").change(function(){
        $(".sports").prop('checked', ($(this).is(':checked')));
    });
    $(".sports").change(function(){
        $("#selectAll").prop('checked', false);
    });
});

edited Sep 14, 2017 at 12:58

Srdjan Dejanovic

1,4292 gold badges20 silver badges33 bronze badges

answered Sep 14, 2017 at 12:37

shetkar vinayak

453 bronze badges

Collectives™ on Stack Overflow

selecting multiple checkbox in jquery

2 Answers 2

8 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

8 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related