jenkins-bot has submitted this change and it was merged.
Change subject: page.py: base isEmptyCategory on cat.categoryinfo
......................................................................
page.py: base isEmptyCategory on cat.categoryinfo
No change of functionality.
Test is based on check of properties rather than try to retrieve
content.
Tests added.
Change-Id: I6070c72882fbcd94a5526a158a9827738386bf51
---
M pywikibot/page.py
M tests/page_tests.py
2 files changed, 13 insertions(+), 3 deletions(-)
Approvals:
John Vandenberg: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/page.py b/pywikibot/page.py
index e77b23e..5d8c8f7 100644
--- a/pywikibot/page.py
+++ b/pywikibot/page.py
@@ -2121,9 +2121,8 @@
def isEmptyCategory(self):
"""Return True if category has no members (including
subcategories)."""
- for member in self.site.categorymembers(self, total=1):
- return False
- return True
+ ci = self.categoryinfo
+ return sum(ci[k] for k in ['files', 'pages', 'subcats'])
== 0
def isHiddenCategory(self):
"""Return True if the category is hidden."""
diff --git a/tests/page_tests.py b/tests/page_tests.py
index ebc37d6..05d3228 100644
--- a/tests/page_tests.py
+++ b/tests/page_tests.py
@@ -400,6 +400,17 @@
# def contributingUsers(self):
+class TestCategoryObject(PywikibotTestCase):
+
+ def test_isEmptyCategory(self):
+ """Test if category is empty or not"""
+ site = pywikibot.Site('en', 'wikipedia')
+ cat_empty = pywikibot.Category(site, u'Category:foooooo')
+ cat_not_empty = pywikibot.Category(site, u'Category:Wikipedia
categories')
+ self.assertTrue(cat_empty.isEmptyCategory())
+ self.assertFalse(cat_not_empty.isEmptyCategory())
+
+
if __name__ == '__main__':
try:
unittest.main()
--
To view, visit
https://gerrit.wikimedia.org/r/157293
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6070c72882fbcd94a5526a158a9827738386bf51
Gerrit-PatchSet: 3
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Mpaa <mpaa.wiki(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: Mpaa <mpaa.wiki(a)gmail.com>
Gerrit-Reviewer: XZise <CommodoreFabianus(a)gmx.de>
Gerrit-Reviewer: jenkins-bot <>