diff options
| author | Tom Christie | 2012-09-06 16:28:11 +0100 |
|---|---|---|
| committer | Tom Christie | 2012-09-06 16:28:11 +0100 |
| commit | 9dc7270cced6be898ff1c3a8855bf196bd89b912 (patch) | |
| tree | 26fbcf87f085964f3fa1ac7ead9c3f3e20f0bc5c /djangorestframework/request.py | |
| parent | b79833ecddcea788b4a8d8901bd25f0afe83bbf7 (diff) | |
| download | django-rest-framework-9dc7270cced6be898ff1c3a8855bf196bd89b912.tar.bz2 | |
Move settings stuff actually into settings
Diffstat (limited to 'djangorestframework/request.py')
| -rw-r--r-- | djangorestframework/request.py | 21 |
1 files changed, 14 insertions, 7 deletions
diff --git a/djangorestframework/request.py b/djangorestframework/request.py index 6c00ff62..5bf7b23c 100644 --- a/djangorestframework/request.py +++ b/djangorestframework/request.py @@ -43,10 +43,9 @@ class Request(object): authenticating the request's user. """ - _USE_FORM_OVERLOADING = True - _METHOD_PARAM = '_method' - _CONTENTTYPE_PARAM = '_content_type' - _CONTENT_PARAM = '_content' + _METHOD_PARAM = api_settings.FORM_METHOD_OVERRIDE + _CONTENT_PARAM = api_settings.FORM_CONTENT_OVERRIDE + _CONTENTTYPE_PARAM = api_settings.FORM_CONTENTTYPE_OVERRIDE def __init__(self, request, parsers=None, authentication=None): self._request = request @@ -194,8 +193,13 @@ class Request(object): form fields or not. """ + USE_FORM_OVERLOADING = ( + self._METHOD_PARAM or + (self._CONTENT_PARAM and self._CONTENTTYPE_PARAM) + ) + # We only need to use form overloading on form POST requests. - if (not self._USE_FORM_OVERLOADING + if (not USE_FORM_OVERLOADING or self._request.method != 'POST' or not is_form_media_type(self._content_type)): return @@ -205,12 +209,15 @@ class Request(object): self._files = self._request.FILES # Method overloading - change the method and remove the param from the content. - if self._METHOD_PARAM in self._data: + if (self._METHOD_PARAM and + self._METHOD_PARAM in self._data): # NOTE: `pop` on a `QueryDict` returns a list of values. self._method = self._data.pop(self._METHOD_PARAM)[0].upper() # Content overloading - modify the content type, and re-parse. - if (self._CONTENT_PARAM in self._data and + if (self._CONTENT_PARAM and + self._CONTENTTYPE_PARAM and + self._CONTENT_PARAM in self._data and self._CONTENTTYPE_PARAM in self._data): self._content_type = self._data.pop(self._CONTENTTYPE_PARAM)[0] self._stream = StringIO(self._data.pop(self._CONTENT_PARAM)[0]) |
