Pywikibot-commits February 2017

pywikibot-commits@lists.wikimedia.org

2 participants
113 discussions

[Gerrit] pywikibot/core[master]: [bugfix] ncTemplates is a method not a property

by jenkins-bot (Code Review)

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/308426 ) Change subject: [bugfix] ncTemplates is a method not a property ...................................................................... [bugfix] ncTemplates is a method not a property Bug: T144690 Change-Id: Iaf31f18a738b21cb59bbbd5da3b47da9e7887d28 --- M scripts/nowcommons.py 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: jenkins-bot: Verified Sn1per: Looks good to me, approved diff --git a/scripts/nowcommons.py b/scripts/nowcommons.py index 62bcd07..04dbc4f 100755 --- a/scripts/nowcommons.py +++ b/scripts/nowcommons.py @@ -51,8 +51,8 @@ # # (C) Wikipedian, 2006-2007 # (C) Siebrand Mazeland, 2007-2008 -# (C) xqt, 2010-2016 -# (C) Pywikibot team, 2006-2016 +# (C) xqt, 2010-2017 +# (C) Pywikibot team, 2006-2017 # # Distributed under the terms of the MIT license. # @@ -455,7 +455,7 @@ self._treat_counter += 1 if not self._treat_counter: pywikibot.output( - 'No transcluded files found for %s.' % self.ncTemplates[0]) + 'No transcluded files found for %s.' % self.ncTemplates()[0]) self.exit() -- To view, visit https://gerrit.wikimedia.org/r/308426 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf31f18a738b21cb59bbbd5da3b47da9e7887d28 Gerrit-PatchSet: 2 Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com> Gerrit-Reviewer: Ladsgroup <Ladsgroup(a)gmail.com> Gerrit-Reviewer: Multichill <maarten(a)mdammers.nl> Gerrit-Reviewer: Sn1per <geofbot(a)gmail.com> Gerrit-Reviewer: jenkins-bot <>

7 years, 2 months

Errored: wikimedia/pywikibot-core#3850 (master - 4536dc2)

by Travis CI

Build Update for wikimedia/pywikibot-core ------------------------------------- Build: #3850 Status: Errored Duration: 22 minutes and 14 seconds Commit: 4536dc2 (master) Author: jenkins-bot Message: Merge "[Bugfix] Fix variable assignment in handleOneLink" View the changeset: https://github.com/wikimedia/pywikibot-core/compare/d93dadad7f46...4536dc2d… View the full build log and details: https://travis-ci.org/wikimedia/pywikibot-core/builds/201480653 -- You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

7 years, 2 months

Errored: wikimedia/pywikibot-core#3849 (master - d93dada)

by Travis CI

Build Update for wikimedia/pywikibot-core ------------------------------------- Build: #3849 Status: Errored Duration: 47 minutes and 13 seconds Commit: d93dada (master) Author: dalba Message: textlib_tests: Don't expect textlib.getCategoryLinks to raise InvalidTitle After fb7f54b6b85f, getCategoryLinks does not raise InvalidTitle anymore, instead a warning is raised. Fix the test according to that change. Bug: T158032 Change-Id: I4777b2ae555837555629b6ac2725d3ba002f36a0 View the changeset: https://github.com/wikimedia/pywikibot-core/compare/2cb6fcdafafa...d93dadad… View the full build log and details: https://travis-ci.org/wikimedia/pywikibot-core/builds/201479788 -- You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

7 years, 2 months

[Gerrit] pywikibot/core[master]: [Bugfix] Fix variable assignment in handleOneLink

by jenkins-bot (Code Review)

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/337433 ) Change subject: [Bugfix] Fix variable assignment in handleOneLink ...................................................................... [Bugfix] Fix variable assignment in handleOneLink Change-Id: I6e6dc467d5f9f9c2cd9a13f109af258e33465ff7 --- M pywikibot/cosmetic_changes.py M tests/cosmetic_changes_tests.py 2 files changed, 11 insertions(+), 10 deletions(-) Approvals: jenkins-bot: Verified Xqt: Looks good to me, approved diff --git a/pywikibot/cosmetic_changes.py b/pywikibot/cosmetic_changes.py index 20fff17..2684d81 100755 --- a/pywikibot/cosmetic_changes.py +++ b/pywikibot/cosmetic_changes.py @@ -526,16 +526,15 @@ firstcase_title = titleWithSection firstcase_label = label - if firstcase_label.startswith(firstcase_title): - if firstcase_label == firstcase_title: - newLink = '[[%s]]' % label - # Check if we can create a link with trailing characters - # instead of a pipelink - elif trailR.sub('', - label[len(titleWithSection):]) == '': - newLink = '[[%s]]%s' % ( - label[:len(titleWithSection)], - label[len(titleWithSection):]) + if firstcase_label == firstcase_title: + newLink = '[[%s]]' % label + # Check if we can create a link with trailing characters + # instead of a pipelink + elif (firstcase_label.startswith(firstcase_title) and + trailR.sub('', label[len(titleWithSection):]) == ''): + newLink = '[[%s]]%s' % ( + label[:len(titleWithSection)], + label[len(titleWithSection):]) else: # Try to capitalize the first letter of the title. diff --git a/tests/cosmetic_changes_tests.py b/tests/cosmetic_changes_tests.py index b5ef83e..6a7112d 100644 --- a/tests/cosmetic_changes_tests.py +++ b/tests/cosmetic_changes_tests.py @@ -247,6 +247,8 @@ def test_cleanUpLinks_pipes(self): """Test cleanUpLinks method.""" + self.assertEqual('[[no|no change]]', + self.cct.cleanUpLinks('[[no|no change]]')) self.assertEqual('[[title]]', self.cct.cleanUpLinks('[[title|title]]')) self.assertEqual('[[title]]', -- To view, visit https://gerrit.wikimedia.org/r/337433 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e6dc467d5f9f9c2cd9a13f109af258e33465ff7 Gerrit-PatchSet: 2 Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-Owner: Matěj Suchánek <matejsuchanek97(a)gmail.com> Gerrit-Reviewer: Dalba <dalba.wiki(a)gmail.com> Gerrit-Reviewer: Magul <tomasz.magulski(a)gmail.com> Gerrit-Reviewer: Matěj Suchánek <matejsuchanek97(a)gmail.com> Gerrit-Reviewer: Urbanecm <martin.urbanec(a)wikimedia.cz> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot <>

7 years, 2 months

[Gerrit] pywikibot/core[master]: textlib_tests: Don't expect textlib.getCategoryLinks to rais...

by jenkins-bot (Code Review)

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/337548 ) Change subject: textlib_tests: Don't expect textlib.getCategoryLinks to raise InvalidTitle ...................................................................... textlib_tests: Don't expect textlib.getCategoryLinks to raise InvalidTitle After fb7f54b6b85f, getCategoryLinks does not raise InvalidTitle anymore, instead a warning is raised. Fix the test according to that change. Bug: T158032 Change-Id: I4777b2ae555837555629b6ac2725d3ba002f36a0 --- M tests/textlib_tests.py 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: jenkins-bot: Verified Xqt: Looks good to me, approved diff --git a/tests/textlib_tests.py b/tests/textlib_tests.py index 2cabf39..f98bb42 100644 --- a/tests/textlib_tests.py +++ b/tests/textlib_tests.py @@ -13,6 +13,10 @@ import functools import os import re +try: + from unittest import mock +except ImportError: + import mock import pywikibot import pywikibot.textlib as textlib @@ -298,8 +302,10 @@ self.assertEqual(textlib.getCategoryLinks( '[[Category:Foo{{!}}and{{!}}bar]]', self.site, expand_text=True), [pywikibot.page.Category(self.site, 'Foo', sortKey='and|bar')]) - self.assertRaises(pywikibot.InvalidTitle, textlib.getCategoryLinks, - '[[Category:nasty{{{!}}]]', self.site) + with mock.patch.object(pywikibot, 'warning', autospec=True) as warn: + textlib.getCategoryLinks('[[Category:nasty{{{!}}]]', self.site) + warn.assert_called_once_with( + 'Invalid category title extracted: nasty{{{!}}') class TestTemplateParams(TestCase): -- To view, visit https://gerrit.wikimedia.org/r/337548 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4777b2ae555837555629b6ac2725d3ba002f36a0 Gerrit-PatchSet: 1 Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-Owner: Dalba <dalba.wiki(a)gmail.com> Gerrit-Reviewer: Huji <huji.huji(a)gmail.com> Gerrit-Reviewer: Mpaa <mpaa.wiki(a)gmail.com> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot <>

7 years, 2 months

Errored: wikimedia/pywikibot-core#3848 (master - 2cb6fcd)

by Travis CI

Build Update for wikimedia/pywikibot-core ------------------------------------- Build: #3848 Status: Errored Duration: 25 minutes and 49 seconds Commit: 2cb6fcd (master) Author: jenkins-bot Message: Merge "archivebot_tests: Fix str2localized_duration tests" View the changeset: https://github.com/wikimedia/pywikibot-core/compare/db71d1ae31a5...2cb6fcda… View the full build log and details: https://travis-ci.org/wikimedia/pywikibot-core/builds/201425620 -- You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

7 years, 2 months

[Gerrit] pywikibot/core[master]: archivebot_tests: Fix str2localized_duration tests

by jenkins-bot (Code Review)

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/337346 ) Change subject: archivebot_tests: Fix str2localized_duration tests ...................................................................... archivebot_tests: Fix str2localized_duration tests 'm' qualifier was recently removed in 236b15c5feae which was causing these test to fail. Replace it with 's'. Keep lines under 80 characters. Bug: T157892 Change-Id: I1875fcf26f57f5af5bc5721c296368706dc3e6d2 --- M tests/archivebot_tests.py 1 file changed, 12 insertions(+), 6 deletions(-) Approvals: jenkins-bot: Verified Whym: Looks good to me, approved diff --git a/tests/archivebot_tests.py b/tests/archivebot_tests.py index 08108f1..aaded1b 100644 --- a/tests/archivebot_tests.py +++ b/tests/archivebot_tests.py @@ -57,20 +57,26 @@ def test_str2localized_duraction_English(self): """Test English localizations of duration.""" site = self.get_site('enwiki') - self.assertEqual(archivebot.str2localized_duration(site, '0m'), '0 minutes') - self.assertEqual(archivebot.str2localized_duration(site, '1d'), '1 day') - self.assertEqual(archivebot.str2localized_duration(site, '10h'), '10 hours') + self.assertEqual( + archivebot.str2localized_duration(site, '0s'), '0 seconds') + self.assertEqual( + archivebot.str2localized_duration(site, '1d'), '1 day') + self.assertEqual( + archivebot.str2localized_duration(site, '10h'), '10 hours') def test_str2localized_duraction_French(self): """Test French localizations of duration.""" site = self.get_site('frwikt') - self.assertEqual(archivebot.str2localized_duration(site, '10d'), '10 jours') - self.assertEqual(archivebot.str2localized_duration(site, '1y'), '1 an') + self.assertEqual( + archivebot.str2localized_duration(site, '10d'), '10 jours') + self.assertEqual( + archivebot.str2localized_duration(site, '1y'), '1 an') def test_str2localized_duraction_Japanese(self): """Test Japanese localizations of duration.""" site = self.get_site('jawiki') - self.assertEqual(archivebot.str2localized_duration(site, '4000m'), '4000 分') + self.assertEqual( + archivebot.str2localized_duration(site, '4000s'), '4000 秒') class TestArchiveBotFunctions(TestCase): -- To view, visit https://gerrit.wikimedia.org/r/337346 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1875fcf26f57f5af5bc5721c296368706dc3e6d2 Gerrit-PatchSet: 4 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: Mpaa <mpaa.wiki(a)gmail.com> Gerrit-Reviewer: Whym <whym(a)whym.org> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot <>

7 years, 2 months

Errored: wikimedia/pywikibot-core#3847 (master - db71d1a)

by Travis CI

Build Update for wikimedia/pywikibot-core ------------------------------------- Build: #3847 Status: Errored Duration: 47 minutes and 42 seconds Commit: db71d1a (master) Author: jenkins-bot Message: Merge "update plural rules" View the changeset: https://github.com/wikimedia/pywikibot-core/compare/f014d625af74...db71d1ae… View the full build log and details: https://travis-ci.org/wikimedia/pywikibot-core/builds/201249261 -- You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

7 years, 2 months

[Gerrit] pywikibot/core[master]: update plural rules

by jenkins-bot (Code Review)

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/307478 ) Change subject: update plural rules ...................................................................... update plural rules from http://localization-guide.readthedocs.io/en/latest/l10n/pluralforms.html Also keep lines beneath 80 chars. Bug: T114978 Change-Id: I50be17b22bb4a661dc527a56d82734318fcf99dd --- M pywikibot/plural.py 1 file changed, 23 insertions(+), 21 deletions(-) Approvals: Mpaa: Looks good to me, approved jenkins-bot: Verified diff --git a/pywikibot/plural.py b/pywikibot/plural.py index ddb6ce1..ddca184 100644 --- a/pywikibot/plural.py +++ b/pywikibot/plural.py @@ -1,8 +1,8 @@ # -*- coding: utf-8 -*- """Module containing plural rules of various languages.""" # -# (C) xqt, 2011 -# (C) Pywikibot team, 2011 +# (C) xqt, 2011-2017 +# (C) Pywikibot team, 2011-2017 # # Distributed under the terms of the MIT license. # @@ -27,13 +27,13 @@ 'ay': {'nplurals': 1, 'plural': 0}, 'be': {'nplurals': 3, 'plural': lambda n: 0 if (n % 10 == 1 and n % 100 != 11) else - 1 if (n % 10 >= 2 and n % 10 <= 4 and (n % 100 < 10 or n % 100 >= 20)) else + 1 if (n % 10 in range(2, 5) and (n % 100 < 10 or n % 100 >= 20)) else 2}, 'bo': {'nplurals': 1, 'plural': 0}, 'br': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'bs': {'nplurals': 3, 'plural': lambda n: 0 if (n % 10 == 1 and n % 100 != 11) else - 1 if (n % 10 >= 2 and n % 10 <= 4 and (n % 100 < 10 or n % 100 >= 20)) else + 1 if (n % 10 in range(2, 5) and (n % 100 < 10 or n % 100 >= 20)) else 2}, 'cgg': {'nplurals': 1, 'plural': 0}, 'cs': {'nplurals': 3, 'plural': lambda n: @@ -42,15 +42,13 @@ 2}, 'csb': {'nplurals': 3, 'plural': lambda n: 0 if (n == 1) else - 1 if (n % 10 >= 2 and n % 10 <= 4 and (n % 100 < 10 or n % 100 >= 20)) else + 1 if n % 10 in range(2, 5) and (n % 100 < 10 or n % 100 >= 20) else 2}, - 'cy': {'nplurals': 6, 'plural': lambda n: - 0 if (n == 0) else - 1 if (n == 1) else - 2 if (n == 2) else - 3 if (n == 3) else - 4 if (n == 6) else - 5}, + 'cy': {'nplurals': 4, 'plural': lambda n: + 0 if (n == 1) else + 1 if (n == 2) else + 2 if (n != 8 and n != 11) else + 3}, 'dz': {'nplurals': 1, 'plural': 0}, 'fa': {'nplurals': 1, 'plural': 0}, 'fil': {'nplurals': 2, 'plural': lambda n: (n > 1)}, @@ -69,9 +67,10 @@ 'gun': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'hr': {'nplurals': 3, 'plural': lambda n: 0 if (n % 10 == 1 and n % 100 != 11) else - 1 if (n % 10 >= 2 and n % 10 <= 4 and (n % 100 < 10 or n % 100 >= 20)) else + 1 if (n % 10 in range(2, 5) and (n % 100 < 10 or n % 100 >= 20)) else 2}, 'id': {'nplurals': 1, 'plural': 0}, + 'is': {'nplurals': 2, 'plural': lambda n: (n % 10 != 1 or n % 100 == 11)}, 'ja': {'nplurals': 1, 'plural': 0}, 'jbo': {'nplurals': 1, 'plural': 0}, 'ka': {'nplurals': 1, 'plural': 0}, @@ -97,7 +96,9 @@ 'mfe': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'mg': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'mi': {'nplurals': 2, 'plural': lambda n: (n > 1)}, - 'mk': {'nplurals': 2, 'plural': lambda n: 0 if n == 1 or n % 10 == 1 else 1}, + 'mk': {'nplurals': 2, 'plural': lambda n: + 0 if n == 1 or n % 10 == 1 else + 1}, 'mnk': {'nplurals': 3, 'plural': lambda n: 0 if (n == 0) else 1 if n == 1 else @@ -108,11 +109,11 @@ 1 if (n == 0 or (n % 100 > 1 and n % 100 < 11)) else 2 if (n % 100 > 10 and n % 100 < 20) else 3}, - 'nso': {'nplurals': 2, 'plural': lambda n: (n > 1)}, + 'my': {'nplurals': 1, 'plural': 0}, 'oc': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'pl': {'nplurals': 3, 'plural': lambda n: 0 if (n == 1) else - 1 if (n % 10 >= 2 and n % 10 <= 4 and (n % 100 < 10 or n % 100 >= 20)) else + 1 if (n % 10 in range(2, 5) and (n % 100 < 10 or n % 100 >= 20)) else 2}, 'pt-br': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'ro': {'nplurals': 3, 'plural': lambda n: @@ -121,8 +122,9 @@ 2}, 'ru': {'nplurals': 3, 'plural': lambda n: 0 if (n % 10 == 1 and n % 100 != 11) else - 1 if (n % 10 >= 2 and n % 10 <= 4 and (n % 100 < 10 or n % 100 >= 20)) else + 1 if (n % 10 in range(2, 5) and (n % 100 < 10 or n % 100 >= 20)) else 2}, + 'sah': {'nplurals': 1, 'plural': 0}, 'sk': {'nplurals': 3, 'plural': lambda n: 0 if (n == 1) else 1 if (n >= 2 and n <= 4) else @@ -134,19 +136,19 @@ 3}, 'sr': {'nplurals': 3, 'plural': lambda n: 0 if (n % 10 == 1 and n % 100 != 11) else - 1 if (n % 10 >= 2 and n % 10 <= 4 and (n % 100 < 10 or n % 100 >= 20)) else + 1 if (n % 10 in range(2, 5) and (n % 100 < 10 or n % 100 >= 20)) else 2}, 'su': {'nplurals': 1, 'plural': 0}, 'th': {'nplurals': 1, 'plural': 0}, 'ti': {'nplurals': 2, 'plural': lambda n: (n > 1)}, - 'tr': {'nplurals': 1, 'plural': 0}, + 'tr': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'tt': {'nplurals': 1, 'plural': 0}, 'ug': {'nplurals': 1, 'plural': 0}, 'uk': {'nplurals': 3, 'plural': lambda n: 0 if (n % 10 == 1 and n % 100 != 11) else - 1 if (n % 10 >= 2 and n % 10 <= 4 and (n % 100 < 10 or n % 100 >= 20)) else + 1 if (n % 10 in range(2, 5) and (n % 100 < 10 or n % 100 >= 20)) else 2}, - 'uz': {'nplurals': 1, 'plural': 0}, + 'uz': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'vi': {'nplurals': 1, 'plural': 0}, 'wa': {'nplurals': 2, 'plural': lambda n: (n > 1)}, 'wo': {'nplurals': 1, 'plural': 0}, -- To view, visit https://gerrit.wikimedia.org/r/307478 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I50be17b22bb4a661dc527a56d82734318fcf99dd Gerrit-PatchSet: 3 Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com> Gerrit-Reviewer: Ladsgroup <Ladsgroup(a)gmail.com> Gerrit-Reviewer: Mpaa <mpaa.wiki(a)gmail.com> Gerrit-Reviewer: jenkins-bot <>

7 years, 2 months

Errored: wikimedia/pywikibot-core#3846 (master - f014d62)

by Travis CI

Build Update for wikimedia/pywikibot-core ------------------------------------- Build: #3846 Status: Errored Duration: 49 minutes and 34 seconds Commit: f014d62 (master) Author: jenkins-bot Message: Merge "[IMPR] Return self when data_repository() is called on DataSite" View the changeset: https://github.com/wikimedia/pywikibot-core/compare/a86fc5737314...f014d625… View the full build log and details: https://travis-ci.org/wikimedia/pywikibot-core/builds/201191160 -- You can configure recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications

7 years, 2 months

← Newer
1
2
3
4
5
6
7
8
9
...
12
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

Pywikibot-commits February 2017