Tricky little jquery selector one : loop over all inputs, and exclude only non-checked radio buttons

Question

Here's the deal : in order to build a correct series of parameters, i'm trying to loop over all inputs included in an ajax built form; Code down here isn't finished, yet I guess you will understand it :

$("#formulaireUploadFichier").live('submit',function(e){
    var fm = $(this);
    var fld = [];
    $('input',fm).not('[type=submit]').each(function(){
        if ($(this"[type=radio]:checked"||$(this).not("[type=radio]"))
        var name= $(this).attr("name");
        var val= $(this).val();
        fld.push(name+'='+val);
    });
    //construct url (note : should really be encoded!)...
    var url = fm.attr('action')+'?'+fld.join('&');
    //open shadowbox...
    Shadowbox.open({
        player:"html",
        content:url,
        width:800,
        height:800
    });
    //prevent form submission...
    e.preventDefault();
}); 

The itchy part comes line 5, where i'm trying to write a correct selector for JQuery to pass over non checked/selected radio buttons. I know i'm not that far from it, but I'm getting messed up, so any help clearing this help would be greatly appreciated.

Answer 1

You've messed up something near $(this"[type

EDIT:

var inps=$('input');
var whatYouWant = inps.filter('[type!=radio]').add(inps.filter('[type=radio]:checked'));

this way it should do what You wanted

Answer 2

What are your expected resutls here?

    $('input',fm).not('[type=submit]').each(function(){
    if ($(this"[type=radio]:checked"||$(this).not("[type=radio]"))
    var name= $(this).attr("name");
    var val= $(this).val();
    fld.push(name+'='+val);
     });

What I see here is that if it's checked or not a radio you want the conditions below to fire. Youre missing thought the {} to surround the code, all youre doing is setting the var name when you meet the condition. Do you mean to have it run all that code? like this?

    $('input',fm).not('[type=submit]').each(function(){
if ($(this"[type=radio]:checked"||$(this).not("[type=radio]")){
var name= $(this).attr("name");
var val= $(this).val();
fld.push(name+'='+val);
}

 });

Answer 3

I'd suggest to replace if ($(this"[type=radio]:checked"||$(this).not("[type=radio]")) by if($(this).attr('type') !== 'radio' || $(this).val()=='on')

Answer 4

Since you're already in a loop of your input elements, no need to start another. Just test for the proper filters on each item.

if($(this).is(":radio:checked,:not(:radio)") {
        var name= $(this).attr("name");
        var val= $(this).val();
        fld.push(name+'='+val);
}

jQuery's is() function lets you compare an element to a selector that you pass as a parameter.

Answer 5

What exactly do you want to select:

• Just checked radio buttons, or
• All checked radio buttons AND checkboxes?

Code for just checked radio buttons:

if($(this).is(":radio:checked")) {

Code for all checked inputs (checkboxes and radio buttons):

if($(this).is(":radio:checked, :not(:radio):checked")) {

I hope this is what you're after.