jenkins-bot has submitted this change and it was merged.

Change subject: [bugfix] Enable -recentchanges without parameters ......................................................................

[bugfix] Enable -recentchanges without parameters

Test added

Bug: T131850 Change-Id: I87e8c27dec105e0b1a404ed4d5e2e2be1e7d442e --- M pywikibot/pagegenerators.py M tests/pagegenerators_tests.py 2 files changed, 9 insertions(+), 1 deletion(-)

Approvals: John Vandenberg: Looks good to me, approved jenkins-bot: Verified

diff --git a/pywikibot/pagegenerators.py b/pywikibot/pagegenerators.py index 0c5f0a2..fdc4be9 100644 --- a/pywikibot/pagegenerators.py +++ b/pywikibot/pagegenerators.py @@ -688,7 +688,7 @@ elif arg == '-recentchanges': rcstart = None rcend = None - params = value.split(',') + params = value.split(',') if value else [] if len(params) == 2: offset = float(params[0]) duration = float(params[1]) diff --git a/tests/pagegenerators_tests.py b/tests/pagegenerators_tests.py index 5adb054..f4f69f8 100755 --- a/tests/pagegenerators_tests.py +++ b/tests/pagegenerators_tests.py @@ -812,6 +812,14 @@ self.assertRaises(ValueError, gf.handleArg, '-recentchanges:3,2,1') self.assertRaises(ValueError, gf.handleArg, '-recentchanges:-10,20')

+ def test_recentchanges_default(self): + """Test recentchanges generator with default setting.""" + gf = pagegenerators.GeneratorFactory(site=self.site) + gf.handleArg('-recentchanges') + gen = gf.getCombinedGenerator() + self.assertIsNotNone(gen) + self.assertPagesInNamespacesAll(gen, set([0, 1, 2]), skip=True) + def test_recentchanges_ns_default(self): """Test recentchanges generator.""" gf = pagegenerators.GeneratorFactory(site=self.site)