diff options
| author | Claude Paroz <claude@2xlibre.net> | 2014-10-20 22:32:43 +0200 |
|---|---|---|
| committer | Claude Paroz <claude@2xlibre.net> | 2014-10-21 09:16:57 +0200 |
| commit | f0bb3c98cc9e128cb0c1622be9eb41a26794c91f (patch) | |
| tree | 9d36849565751d4310e7a10754483b9f697db208 /django | |
| parent | edcb33c92e11ad5ac92a928e65b7d321fa9d067c (diff) | |
Fixed #21740 -- Allowed test client data to be an empty string
This fixes a regression introduced by 2a31d00933.
Thanks tony-zhu for the report.
Diffstat (limited to 'django')
| -rw-r--r-- | django/test/client.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/django/test/client.py b/django/test/client.py index a3a8f21a63..b1ff33e3d3 100644 --- a/django/test/client.py +++ b/django/test/client.py @@ -279,8 +279,9 @@ class RequestFactory(object): def get(self, path, data=None, secure=False, **extra): "Construct a GET request." + data = {} if data is None else data r = { - 'QUERY_STRING': urlencode(data or {}, doseq=True), + 'QUERY_STRING': urlencode(data, doseq=True), } r.update(extra) return self.generic('GET', path, secure=secure, **r) @@ -289,7 +290,8 @@ class RequestFactory(object): secure=False, **extra): "Construct a POST request." - post_data = self._encode_data(data or {}, content_type) + data = {} if data is None else data + post_data = self._encode_data(data, content_type) return self.generic('POST', path, post_data, content_type, secure=secure, **extra) @@ -297,8 +299,9 @@ class RequestFactory(object): def head(self, path, data=None, secure=False, **extra): "Construct a HEAD request." + data = {} if data is None else data r = { - 'QUERY_STRING': urlencode(data or {}, doseq=True), + 'QUERY_STRING': urlencode(data, doseq=True), } r.update(extra) return self.generic('HEAD', path, secure=secure, **r) |