Passing values into django forms

Question

I have googled around trying to figure out and understand how this works, yet I still haven't grasped this quite right. What I want to do is pass a value into a form to then use for a query. I have a session variable called menu_term, which determines the choices in the form.

from views.py

def manage_groups(request):
    form = CourseGroupForm(request,current_term=request.session.get('menu_term'))
return render_to_response("accounts/group_management.html", {'form':form}, context_instance=RequestContext(request))

from forms.py

 class CourseGroupForm(ModelForm):
 def __init__(self, current_term, *args, **kwargs):
    super(CourseGroupForm, self).__init__(*args, **kwargs)
    courseslist = Course.objects.filter(term=current_term, num_in=settings.LAB_COURSES).order_by('description').distinct();
    print(courseslist)
    self.fields['courses'].queryset = forms.ChoiceField(label='Select Course', choices=courseslist)

class Meta:
    model = CourseGroup
    fields = ['name','courses'];

The error I am getting is:

 __init__() got multiple values for keyword argument 'current_term'

For the benefit of anyone else coming across this, what are the proper ways of defining a form that takes a value passed in from outside?

Thanks, Good Day MJ

dm03514 · Accepted Answer · 2015-02-05 14:08:53Z

5

Its important to pop the kwarg you instantiate your form with before calling the forms super __init__

class CourseGroupForm(ModelForm):
 def __init__(self, current_term, *args, **kwargs):
    current_term = kwargs.pop('current_term')
    super(CourseGroupForm, self).__init__(*args, **kwargs)

The above assumes current_term is always present.

as @vishen pointed out, check your arguments, you are initializing your form with request as the value for current_term

edited Feb 5, 2015 at 14:08

answered Feb 5, 2015 at 0:09

dm03514

56.2k18 gold badges117 silver badges147 bronze badges

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

2 Comments

bmhkim Over a year ago

You can also call 'del' on the kwarg: "del kwargs['current_term']" to remove it.

Alvaro Over a year ago

Also, point out that this line "self.fields['courses'].queryset = forms.ChoiceField(label='Select Course', choices=courseslist)" is wrong. He's assigning a field where a queryset should be given

vishen · Accepted Answer · 2015-02-05 00:50:37Z

2

The error is happening because in your model form init decleration

 class CourseGroupForm(ModelForm):
     def __init__(self, current_term, *args, **kwargs)

current_term is the first argument that the form is expecting to find, but because you are passing through the request object first and then the current_term after that, your effiectely passing the following

form = CourseGroupForm(current_term=request,current_term=request.session.get('menu_term'))

Hence the multiple values for keyword argument 'current_term' error message.

answered Feb 5, 2015 at 0:50

vishen

4692 silver badges7 bronze badges

Collectives™ on Stack Overflow

Passing values into django forms

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related