jenkins-bot has submitted this change and it was merged.
Change subject: Dont pass None as api param value
......................................................................
Dont pass None as api param value
Change-Id: I43cb53e4c61182031ef25a14589cd10b1f799bbd
---
M pywikibot/data/api.py
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
John Vandenberg: Looks good to me, but someone else must approve
XZise: Looks good to me, but someone else must approve
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/data/api.py b/pywikibot/data/api.py
index 741a75b..7f4210e 100644
--- a/pywikibot/data/api.py
+++ b/pywikibot/data/api.py
@@ -996,8 +996,8 @@
value = values[0]
if value is True:
values = ['']
- elif value is False:
- # False booleans are not in the http URI
+ elif value is False or value is None:
+ # False and None are not included in the http URI
continue
iterator = iter(values)
value = u'|'.join(self._format_value(value) for value in iterator)
--
To view, visit
https://gerrit.wikimedia.org/r/197813
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I43cb53e4c61182031ef25a14589cd10b1f799bbd
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: XZise <CommodoreFabianus(a)gmx.de>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>