summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Cordasco <sigmavirus24@users.noreply.github.com>2015-08-15 10:39:51 -0500
committerIan Cordasco <sigmavirus24@users.noreply.github.com>2015-08-15 10:39:51 -0500
commitba32c6871cda62c7e46961bab117d2d0f88fbf33 (patch)
tree560d00179e42f37fd7c94c536b574c997ed13686
parent53ea23128e375ef84511fd2b22b740eca15d0854 (diff)
downloadpython-requests-revert-2706-fix-merge-setting.tar.gz
Revert "Fix merge setting for not preserving original order of dict parameters"revert-2706-fix-merge-setting
-rw-r--r--requests/sessions.py4
-rwxr-xr-xtest_requests.py7
2 files changed, 3 insertions, 8 deletions
diff --git a/requests/sessions.py b/requests/sessions.py
index 7c754608..50291fac 100644
--- a/requests/sessions.py
+++ b/requests/sessions.py
@@ -63,10 +63,12 @@ def merge_setting(request_setting, session_setting, dict_class=OrderedDict):
merged_setting.update(to_key_val_list(request_setting))
# Remove keys that are set to None.
- for (k, v) in merged_setting.items():
+ for (k, v) in request_setting.items():
if v is None:
del merged_setting[k]
+ merged_setting = dict((k, v) for (k, v) in merged_setting.items() if v is not None)
+
return merged_setting
diff --git a/test_requests.py b/test_requests.py
index 7e5e4d8f..a5d63464 100755
--- a/test_requests.py
+++ b/test_requests.py
@@ -125,13 +125,6 @@ class RequestsTestCase(unittest.TestCase):
"http://example.com/path?key=value#fragment", params={"a": "b"}).prepare()
assert request.url == "http://example.com/path?key=value&a=b#fragment"
- def test_params_original_order_is_preserved_by_default(self):
- param_ordered_dict = collections.OrderedDict((('z', 1), ('a', 1), ('k', 1), ('d', 1)))
- session = requests.Session()
- request = requests.Request('GET', 'http://example.com/', params=param_ordered_dict)
- prep = session.prepare_request(request)
- assert prep.url == 'http://example.com/?z=1&a=1&k=1&d=1'
-
def test_mixed_case_scheme_acceptable(self):
s = requests.Session()
s.proxies = getproxies()