jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/306082 )
Change subject: Site.search: Search all namespaces when none is given
......................................................................
Site.search: Search all namespaces when none is given
This behavior matches what had been previously described in the docstring.
Also remove the warning which was falsely triggered when the namespaces was 0.
Amend the related test accordingly.
Change-Id: I49df0a1441ec976a474a3d05448b4d09ea932a60
---
M pywikibot/site.py
M tests/site_tests.py
2 files changed, 2 insertions(+), 5 deletions(-)
Approvals:
Lokal Profil: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/site.py b/pywikibot/site.py
index 31cbebd..c317f8c 100644
--- a/pywikibot/site.py
+++ b/pywikibot/site.py
@@ -4498,11 +4498,8 @@
where = 'titles'
else:
where = 'title'
- if namespaces == []:
+ if not namespaces and namespaces != 0:
namespaces = [ns_id for ns_id in self.namespaces if ns_id >= 0]
- if not namespaces:
- pywikibot.warning(u"search: namespaces cannot be empty; using
[0].")
- namespaces = [0]
srgen = self._generator(api.PageGenerator, type_arg="search",
gsrsearch=searchstring, gsrwhat=where,
namespaces=namespaces,
diff --git a/tests/site_tests.py b/tests/site_tests.py
index 82f1c0b..d95e41f 100644
--- a/tests/site_tests.py
+++ b/tests/site_tests.py
@@ -1377,7 +1377,7 @@
"""Test the site.search() method."""
mysite = self.site
try:
- se = list(mysite.search("wiki", total=100))
+ se = list(mysite.search("wiki", total=100, namespaces=0))
self.assertLessEqual(len(se), 100)
self.assertTrue(all(isinstance(hit, pywikibot.Page)
for hit in se))
--
To view, visit
https://gerrit.wikimedia.org/r/306082
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I49df0a1441ec976a474a3d05448b4d09ea932a60
Gerrit-PatchSet: 5
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: Lokal Profil <lokal.profil(a)gmail.com>
Gerrit-Reviewer: Merlijn van Deen <valhallasw(a)arctus.nl>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>