[Pywikibot-commits] [Gerrit] pywikibot/core[master]: Request._add_defaults: Make sure the order of parameters doe...

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 &lt;dalba.wiki(a)gmail.com&gt; Gerrit-Reviewer: Dalba &lt;dalba.wiki(a)gmail.com&gt; Gerrit-Reviewer: John Vandenberg &lt;jayvdb(a)gmail.com&gt; Gerrit-Reviewer: Magul &lt;tomasz.magulski(a)gmail.com&gt; Gerrit-Reviewer: Mpaa &lt;mpaa.wiki(a)gmail.com&gt; Gerrit-Reviewer: Xqt &lt;info(a)gno.de&gt; Gerrit-Reviewer: jenkins-bot <>