jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/344584 )
Change subject: Request._add_defaults: Make sure the order of parameters does not change
......................................................................
Request._add_defaults: Make sure the order of parameters does not change
Using `set` on parameters was causing occasional cache misses because the
order of elements in a set is not guaranteed to be preserved.
Bug: T161291
Change-Id: I6ff4ddacf31af2d7ac1535b98dbeb4a5e46cd47a
---
M pywikibot/data/api.py
1 file changed, 5 insertions(+), 5 deletions(-)
Approvals:
jenkins-bot: Verified
Xqt: Looks good to me, approved
diff --git a/pywikibot/data/api.py b/pywikibot/data/api.py
index 9030e4b..4f362af 100644
--- a/pywikibot/data/api.py
+++ b/pywikibot/data/api.py
@@ -1,7 +1,7 @@
# -*- coding: utf-8 -*-
"""Interface to Mediawiki's api.php."""
#
-# (C) Pywikibot team, 2007-2016
+# (C) Pywikibot team, 2007-2017
#
# Distributed under the terms of the MIT license.
#
@@ -1683,14 +1683,14 @@
meta = self._params.get("meta", [])
if "userinfo" not in meta:
meta = set(meta + ['userinfo'])
- self._params['meta'] = list(meta)
+ self._params['meta'] = sorted(meta)
uiprop = self._params.get("uiprop", [])
uiprop = set(uiprop + ["blockinfo", "hasmsg"])
- self._params["uiprop"] = list(sorted(uiprop))
+ self._params['uiprop'] = sorted(uiprop)
if 'prop' in self._params:
if self.site.has_extension('ProofreadPage'):
prop = set(self._params['prop'] + ['proofread'])
- self._params['prop'] = list(prop)
+ self._params['prop'] = sorted(prop)
# When neither 'continue' nor 'rawcontinue' is present and
the
# version number is at least 1.25wmf5 we add a dummy rawcontinue
# parameter. Querying siteinfo is save as it adds 'continue'.
@@ -2322,7 +2322,7 @@
% (self.__class__.__name__, filename, uniquedescr),
_logger)
return True
- except IOError as e:
+ except IOError:
# file not found
return False
except Exception as e:
--
To view, visit
https://gerrit.wikimedia.org/r/344584
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6ff4ddacf31af2d7ac1535b98dbeb4a5e46cd47a
Gerrit-PatchSet: 7
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Dalba <dalba.wiki(a)gmail.com>
Gerrit-Reviewer: Dalba <dalba.wiki(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Magul <tomasz.magulski(a)gmail.com>
Gerrit-Reviewer: Mpaa <mpaa.wiki(a)gmail.com>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>