views:

63

answers:

2

Hi there, i have a simple view in wich i'm saving a form. the code seems 'clean',but i can't get rid of the error : "The view didn't return an HttpResponse object." Though i've searched on the web, i did not find a relevant indication.

def classroom_privacy(request,classname):
         theclass = Classroom.objects.get(classname=classname)
     if request.method == 'POST':  
       form = PrivacyClass(request.POST)
       if form.is_valid():
           new_obj = form.save(commit=False)
           new_obj.save()
           return HttpResponseRedirect('.')    
       else:
           form = PrivacyClass()     
       return render_to_response('classroom/classroom_privacy.html', {'form': form}, 
          context_instance=RequestContext(request))  

Thanks!

+1  A: 

All view functions must return some kind of HttpResponse object. There exists a code path in your function where None will be returned instead. This will occur when request.method != 'POST' and you'll simply "fall off the end" of your function (which will return None).

Brian Neal
yes. you are right. my problem was more trivial yet, though i couldn't see. i had an indentation problem, and that 'else' was interpreted as being for the second if.Thanks so much for interest! :)
dana
+4  A: 

verify the indentation of your code

def classroom_privacy(request, classname):
    theclass = Classroom.objects.get(classname=classname)
    if request.method == 'POST':
        form = PrivacyClass(request.POST)
        if form.is_valid():
            new_obj = form.save(commit=False)
            new_obj.save()
            return HttpResponseRedirect('.') 
    else:
        form = PrivacyClass()  

    return render_to_response('classroom/classroom_privacy.html', {'form': form}, context_instance=RequestContext(request))

if it is get request, render a unbound form

if it is post request and invalid form render a bound form

if it is post request and valid form redirect the page

Ashok
Absolutely correct!
KevinDTimm
you're right. I actually had am indentation problem ! thank you!
dana