4

I am a beginner in jquery so please bear with me. I have a jquery function that allows me to select multiple checkboxes and create a string as follows:

function getSelectedVals(){
     var tmp =[];
     $("input[name='checks']").each(function() {
     if ($(this).attr('checked'))
     {
        checked = ($(this).val());
        tmp.push(checked);
     }
     });
     var filters = tmp.join(',');
     alert(filters)
     return filters;
}

I then call a django view function and pass the string as follows:

selected = getSelectedVals();
var myurl = "/bills/delete/?id=" + selected;
$.ajax({
    type: "GET",
    url: myurl,
    data: selected,
    cache: false

});

On the server I have a delete view function that iterates over the checkbox values and manipulates a list.

def delete(request):
    global myarray
    idx = request.GET[u'id']
    listidx = idx.split(',')
    for l in listidx:
        value = myarray[int(l)]
        myarray.remove(value)
    return HttpResponse("/bills/jqtut/")

The problem is that on the server all the indexes I am sending as the GET string are not being handled, only half are.

Please help me! Thanks

1
  • You can shorten your each function using $("input[name='checks']:checked").each and remove the if statement. Commented Feb 3, 2010 at 10:40

2 Answers 2

9

From what I see, you do it the way around. You should set the same name on all checkboxes. I don't know why do you send it by GET, I'd suggest sending it by POST.

<input type="checkbox" name="vehicle" value="Bike" />
<input type="checkbox" name="vehicle" value="Car" />
<input type="checkbox" name="vehicle" value="Airplane" />

Then, use getlist() method in your view:

def delete(request):
    values = request.POST.getlist(u'vehicle')
    # Handling goes here.

But anyway, unless you really need to do custom stuff (however special cases are not special enough to break the rules ;), use Django forms. There already is a checkbox list OOTB. I'm not sure why do you even consider using JavaScript in this very case.

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

1 Comment

Thanks ! However I am stuck again:( I am trying to send "filters" in the above code as POST data to Django , but I cannot figure out the correct dataType. What is the correct way to post this data so that django sees the list which it can iterate ?
2

firstly, I don't know if you need to manipulate the checkbox values into a string like you are doing. if you allocate them the same name attribute, http (or jQuery's) serialisation will do it for you.

On the Django side use getlist to grab a list, the [] accessor only gets the last value in a list of params.

Http default handling however doesn't send through the values of checkboxes that aren't checked, so you might want to build in some checks that compare the checkboxes you build vs the data you get back.

As Gavoja says above, django forms is probably a good answer for this: Specifically, you want to use MultipleChoiceField with a CheckboxSelectMultiple widget and the choices as your options:

my_field = forms.MultipleChoiceField(choices=SOME_CHOICES, widget=forms.CheckboxSelectMultiple())

also see: In Django is there a way to display choices as checkboxes?

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.