diff options
author | Ian Cordasco <sigmavirus24@users.noreply.github.com> | 2015-08-15 10:39:51 -0500 |
---|---|---|
committer | Ian Cordasco <sigmavirus24@users.noreply.github.com> | 2015-08-15 10:39:51 -0500 |
commit | ba32c6871cda62c7e46961bab117d2d0f88fbf33 (patch) | |
tree | 560d00179e42f37fd7c94c536b574c997ed13686 | |
parent | 53ea23128e375ef84511fd2b22b740eca15d0854 (diff) | |
download | python-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.py | 4 | ||||
-rwxr-xr-x | test_requests.py | 7 |
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() |