XZise has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/232303

Change subject: [FIX] cache: Wrap items into a list
......................................................................

[FIX] cache: Wrap items into a list

With 954825e1 the Request class expects that each URL parameter contains a list
to simplify the handling of it. With eedef173 the maintenance script itself
used the new `_params` variable but wasn't actually converting the entries into
lists so that the Request class thought (again due to 954825e1) the strings
itself were lists and thus joined each character.

Change-Id: Ie36a9adcb46675b576226d92e0b2c483b7e2f0d5
---
M scripts/maintenance/cache.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core 
refs/changes/03/232303/1

diff --git a/scripts/maintenance/cache.py b/scripts/maintenance/cache.py
index 7669f17..6478c7a 100755
--- a/scripts/maintenance/cache.py
+++ b/scripts/maintenance/cache.py
@@ -171,7 +171,7 @@
             self.site._username = [username, username]
         if not params:
             raise ParseError('No request params')
-        self._params = dict(eval(params))
+        self._params = dict((key, [value]) for key, value in eval(params))
 
     def _delete(self):
         """Delete the cache entry."""

-- 
To view, visit https://gerrit.wikimedia.org/r/232303
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie36a9adcb46675b576226d92e0b2c483b7e2f0d5
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: XZise <commodorefabia...@gmx.de>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to