DRYer instantiation of Forms

1
2
3
4
5
6
def form_kwargs(request):
    kwargs = {}
    if request.method == 'POST':
        kwargs['data'] = request.POST
        kwargs['files'] = request.FILES
    return kwargs

More like this

  1. Hidden Forms by insin 4 years, 10 months ago
  2. ImageField for admin with thumbnail by semente 2 years, 11 months ago
  3. SignedForm: CSRF-protect forms with a hidden token field by exogen 3 years, 8 months ago
  4. Form row filter by kylefox 4 years, 3 months ago
  5. A Lazy ChoiceField implementation by lsbardel 2 years, 7 months ago

Comments

mark0978 (on April 21, 2009):

Maybe its just me, but I don't see how this is "DRY"er. Not even sure what the point of it is.

#

SmileyChris (on May 6, 2009):

Normally, you'd write:

if request.method == 'POST':
    form = MyForm(data=request.POST, files=request.FILES)
    if form.is_valid():
         form.save()
         # Redirect
else:
    form = MyForm()

Now you write:

form = MyForm(**form_kwargs(request))
if form.is_valid():
     form.save()
     # Redirect

#

SmileyChris (on April 25, 2010):

PS: if you'd rather not bother with a helper like this, the method of instanciating a form only once for both cases is still useful.

The most simple case is (where files aren't involved and you always expect at least one thing in your POST):

form = MyForm(data=request.POST or None)
if form.is_valid():
    form.save()
    # Redirect

#

(Forgotten your password?)