"if request.POST" should read "if request.method == 'POST'"

[davidschein@…]

From ​http://www.djangoproject.com/documentation/request_response/#httprequest-objects
we have:

It's possible that a request can come in via POST with an empty POST dictionary -- if, say, a form is requested via the POST HTTP method but does not include form data. Therefore, you shouldn't use if request.POST to check for use of the POST method; instead, use if request.method == "POST" (see above).

Then in ​http://www.djangoproject.com/documentation/forms/
we violate that advice with:

def create_place(request):
    manipulator = Place.AddManipulator()

    if request.POST:
    ...

[Malcolm Tredinnick]

(In [3914]) Fixed #2931 -- Use request.method == 'POST' where appropriate in the examples.
Thanks, David Schein.

[Gary Wilson <gary.wilson@…>]

request.POST is used once elsewhere in the docs and several places in trunk too that maybe should be changed.

[Bastian Kleineidam <calvin@…>]

Hmm, should that be request.method.lower() == 'post' to be really sure? Just nitpicking, though.

[Chris Beaven]

.lower() shouldn't be necessary: according to documentation, request.method will always return an uppercase value.

[Adrian Holovaty]

(In [4110]) Fixed #2931 -- Changed 'if request.POST' to 'if request.method == POST' in docs/sessions.txt