Pywikibot-commits January 2020

pywikibot-commits@lists.wikimedia.org

1 participants
95 discussions

[Gerrit] pywikibot/core[master]: [bugfix] Fix parameters in OauthLoginManager()
by jenkins-bot (Code Review) 31 Jan '20

31 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/569101 ) Change subject: [bugfix] Fix parameters in OauthLoginManager() ...................................................................... [bugfix] Fix parameters in OauthLoginManager() Fix parameters when calling OauthLoginManager(), after removal of sysop parameter. Change-Id: I56ede124fe2f6f3add26a315f22b127cd34e66a2 --- M scripts/login.py M tests/oauth_tests.py 2 files changed, 2 insertions(+), 3 deletions(-) Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified diff --git a/scripts/login.py b/scripts/login.py index cfabfb6..e90f814 100755 --- a/scripts/login.py +++ b/scripts/login.py @@ -80,8 +80,7 @@ def _oauth_login(site): consumer_key, consumer_secret = _get_consumer_token(site) - login_manager = OauthLoginManager(consumer_secret, False, site, - consumer_key) + login_manager = OauthLoginManager(consumer_secret, site, consumer_key) login_manager.login() identity = login_manager.identity if identity is None: diff --git a/tests/oauth_tests.py b/tests/oauth_tests.py index cc44abd..00cbdee 100644 --- a/tests/oauth_tests.py +++ b/tests/oauth_tests.py @@ -79,7 +79,7 @@ self.assertIsNotNone(login_manager.access_token) self.assertIsInstance(login_manager.identity, dict) self.assertEqual(login_manager.identity['username'], - self.site.username(sysop=False)) + self.site.username()) if __name__ == '__main__': # pragma: no cover -- To view, visit https://gerrit.wikimedia.org/r/569101 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I56ede124fe2f6f3add26a315f22b127cd34e66a2 Gerrit-Change-Number: 569101 Gerrit-PatchSet: 1 Gerrit-Owner: Mpaa <mpaa.wiki(a)gmail.com> Gerrit-Reviewer: D3r1ck01 <xsavitar.wiki(a)aol.com> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: [IMPR] Use wikipedia family for testwiki
by jenkins-bot (Code Review) 31 Jan '20

31 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/569096 ) Change subject: [IMPR] Use wikipedia family for testwiki ...................................................................... [IMPR] Use wikipedia family for testwiki Change-Id: I336e5a2253b2cc72fb11b3dd6052f8e832a5ad93 --- M pywikibot/pagegenerators.py M tests/replacebot_tests.py M tox.ini 3 files changed, 5 insertions(+), 4 deletions(-) Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified diff --git a/pywikibot/pagegenerators.py b/pywikibot/pagegenerators.py index d524fe0..25db7bb 100644 --- a/pywikibot/pagegenerators.py +++ b/pywikibot/pagegenerators.py @@ -3062,7 +3062,8 @@ } # test wikipedia - if self.site.code == 'test' and self.site.family == 'test': + if (self.site.code == 'test' + and self.site.family in ['test', 'wikipedia']): query['language'] = 'test' query['project'] = 'wikipedia' diff --git a/tests/replacebot_tests.py b/tests/replacebot_tests.py index c533495..e1a461b 100644 --- a/tests/replacebot_tests.py +++ b/tests/replacebot_tests.py @@ -90,7 +90,7 @@ """Run the L{replace.main} with the given args and summary and page.""" # -page to not have an empty generator # -lang and -family as it will use Site() otherwise - return replace.main(*(args + ('-lang:test', '-family:test', + return replace.main(*(args + ('-lang:test', '-family:wikipedia', '-page:TEST'))) def test_invalid_replacements(self): diff --git a/tox.ini b/tox.ini index 15f9ca7..8a31790 100644 --- a/tox.ini +++ b/tox.ini @@ -28,7 +28,7 @@ flake8: flake8 --doctests {posargs} test: python setup.py test - fasttest: python -W error::UserWarning -m generate_user_files -family:test -lang:test -v + fasttest: python -W error::UserWarning -m generate_user_files -family:wikipedia -lang:test -v fasttest: nosetests --version fasttest: nosetests --with-detecthttp -v -a '!net' tests deps = @@ -53,7 +53,7 @@ [testenv:doctest] commands = - python -W error::UserWarning -m generate_user_files -family:test -lang:test -v + python -W error::UserWarning -m generate_user_files -family:wikipedia -lang:test -v nosetests --version nosetests --with-doctest pywikibot {[params]doctest_skip} basepython = -- To view, visit https://gerrit.wikimedia.org/r/569096 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I336e5a2253b2cc72fb11b3dd6052f8e832a5ad93 Gerrit-Change-Number: 569096 Gerrit-PatchSet: 4 Gerrit-Owner: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: [IMPR] use a list for tests in test_valid
by jenkins-bot (Code Review) 31 Jan '20

31 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/567468 ) Change subject: [IMPR] use a list for tests in test_valid ...................................................................... [IMPR] use a list for tests in test_valid - use extended_title_tests list for all remaining tests - use subTest for every test in this list Change-Id: I75399c0b0e6d95b4a7cafd23d07cdbf3f959e763 --- M tests/link_tests.py 1 file changed, 23 insertions(+), 16 deletions(-) Approvals: Mpaa: Looks good to me, approved jenkins-bot: Verified diff --git a/tests/link_tests.py b/tests/link_tests.py index 1307ac8..548ef0c 100644 --- a/tests/link_tests.py +++ b/tests/link_tests.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """Test Link functionality.""" # -# (C) Pywikibot team, 2014-2019 +# (C) Pywikibot team, 2014-2020 # # Distributed under the terms of the MIT license. # @@ -71,27 +71,34 @@ def test_valid(self): """Test that valid titles are correctly normalized.""" - site = self.get_site() title_tests = ['Sandbox', 'A "B"', "A 'B'", '.com', '~', '"', "'", 'Foo/.../Sandbox', 'Sandbox/...', 'A~~', 'X' * 252] + + extended_title_tests = [ + ('Talk:Sandbox', 'Sandbox'), + ('Talk:Foo:Sandbox', 'Foo:Sandbox'), + ('File:Example.svg', 'Example.svg'), + ('File_talk:Example.svg', 'Example.svg'), + (':A', 'A'), + # Length is 256 total, but only title part matters + ('Category:' + 'X' * 248, 'X' * 248), + ('A%20B', 'A B'), + ('A é B', 'A é B'), + ('A é B', 'A é B'), + ('A é B', 'A é B'), + ('A   B', 'A B'), + ('A   B', 'A B'), + ] + + site = self.get_site() + for title in title_tests: with self.subTest(title=title): self.assertEqual(Link(title, site).title, title) - self.assertEqual(Link('Talk:Sandbox', site).title, 'Sandbox') - self.assertEqual(Link('Talk:Foo:Sandbox', site).title, 'Foo:Sandbox') - self.assertEqual(Link('File:Example.svg', site).title, 'Example.svg') - self.assertEqual(Link('File_talk:Example.svg', site).title, - 'Example.svg') - self.assertEqual(Link(':A', site).title, 'A') - # Length is 256 total, but only title part matters - self.assertEqual(Link('Category:' + 'X' * 248, site).title, 'X' * 248) - self.assertEqual(Link('A%20B', site).title, 'A B') - self.assertEqual(Link('A é B', site).title, 'A é B') - self.assertEqual(Link('A é B', site).title, 'A é B') - self.assertEqual(Link('A é B', site).title, 'A é B') - self.assertEqual(Link('A   B', site).title, 'A B') - self.assertEqual(Link('A   B', site).title, 'A B') + for link, title in extended_title_tests: + with self.subTest(link=link, title=title): + self.assertEqual(Link(link, site).title, title) anchor_link = Link('A | B', site) self.assertEqual(anchor_link.title, 'A') -- To view, visit https://gerrit.wikimedia.org/r/567468 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I75399c0b0e6d95b4a7cafd23d07cdbf3f959e763 Gerrit-Change-Number: 567468 Gerrit-PatchSet: 2 Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: Mpaa <mpaa.wiki(a)gmail.com> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: [bugfix] remove sysop assignment in Site interfaces
by jenkins-bot (Code Review) 31 Jan '20

31 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/568504 ) Change subject: [bugfix] remove sysop assignment in Site interfaces ...................................................................... [bugfix] remove sysop assignment in Site interfaces Bug: T243906 Bug: T71283 Change-Id: Ief5e4b904e1789484027c127d868104b8146b4d8 --- M pywikibot/__init__.py M tests/family_tests.py M tests/site_detect_tests.py M tests/utils.py 4 files changed, 5 insertions(+), 7 deletions(-) Approvals: Dvorapa: Looks good to me, but someone else must approve Mpaa: Looks good to me, approved jenkins-bot: Verified diff --git a/pywikibot/__init__.py b/pywikibot/__init__.py index 5e33a21..cbb6ff3 100644 --- a/pywikibot/__init__.py +++ b/pywikibot/__init__.py @@ -1263,7 +1263,7 @@ user = normalize_username(user) key = '%s:%s:%s:%s' % (interface.__name__, fam, code, user) if key not in _sites or not isinstance(_sites[key], interface): - _sites[key] = interface(code=code, fam=fam, user=user, sysop=None) + _sites[key] = interface(code=code, fam=fam, user=user) debug("Instantiated %s object '%s'" % (interface.__name__, _sites[key]), _logger) diff --git a/tests/family_tests.py b/tests/family_tests.py index d33c2a8..8c321a1 100644 --- a/tests/family_tests.py +++ b/tests/family_tests.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """Tests for the family module.""" # -# (C) Pywikibot team, 2014-2019 +# (C) Pywikibot team, 2014-2020 # # Distributed under the terms of the MIT license. # @@ -178,7 +178,7 @@ self.assertEqual(kwargs, {}) self.assertEqual(code, self.current_code) self.assertEqual(fam, self.current_family) - site = DrySite(code, fam, None, None) + site = DrySite(code, fam, None) site._siteinfo._cache['general'] = ({'articlepath': self.article_path}, True) return site diff --git a/tests/site_detect_tests.py b/tests/site_detect_tests.py index a2fddc2..d1a1c04 100644 --- a/tests/site_detect_tests.py +++ b/tests/site_detect_tests.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """Test for site detection.""" # -# (C) Pywikibot team, 2014-2019 +# (C) Pywikibot team, 2014-2020 # # Distributed under the terms of the MIT license. # @@ -339,8 +339,6 @@ self.assertEqual(code, fam.code) self.assertEqual(fam.domain, self.NETLOC) self.assertEqual(user, self.USERNAME) - if not args and 'sysop' not in kwargs: - kwargs['sysop'] = None site = DrySite(code, fam, user, *args, **kwargs) site._siteinfo._cache.update( (key, (value, True)) diff --git a/tests/utils.py b/tests/utils.py index f2536bc..56e5f09 100644 --- a/tests/utils.py +++ b/tests/utils.py @@ -365,7 +365,7 @@ _loginstatus = pywikibot.site.LoginStatus.NOT_ATTEMPTED - def __init__(self, code, fam, user, sysop): + def __init__(self, code, fam, user): """Initializer.""" super(DrySite, self).__init__(code, fam, user) self._userinfo = pywikibot.tools.EMPTY_DEFAULT -- To view, visit https://gerrit.wikimedia.org/r/568504 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ief5e4b904e1789484027c127d868104b8146b4d8 Gerrit-Change-Number: 568504 Gerrit-PatchSet: 4 Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: Mpaa <mpaa.wiki(a)gmail.com> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: [cleanup] Remove deprecated/not implemented rcstream methods
by jenkins-bot (Code Review) 30 Jan '20

30 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/568982 ) Change subject: [cleanup] Remove deprecated/not implemented rcstream methods ...................................................................... [cleanup] Remove deprecated/not implemented rcstream methods rcstream support was removed with 3.0.20171212 in favour of eventstreams. The old path methods are obsolete. - remove rcstream* methods from Family - remove rcstream_host from WikimediaFamily Change-Id: I5b5c93cc6b1fe5a6b3554d160ac50522ab8e550d --- M pywikibot/family.py 1 file changed, 1 insertion(+), 18 deletions(-) Approvals: Dvorapa: Looks good to me, approved jenkins-bot: Verified diff --git a/pywikibot/family.py b/pywikibot/family.py index 9eeccc6..abe13db 100644 --- a/pywikibot/family.py +++ b/pywikibot/family.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """Objects representing MediaWiki families.""" # -# (C) Pywikibot team, 2004-2019 +# (C) Pywikibot team, 2004-2020 # # Distributed under the terms of the MIT license. # @@ -1217,18 +1217,6 @@ """DEPRECATED: Return nice path prefix, e.g. '/wiki/'.""" return '/wiki/' - def rcstream_host(self, code): - """Hostname for RCStream.""" - raise NotImplementedError('This family does not support RCStream') - - def rcstream_path(self, code): - """Return path for RCStream.""" - raise NotImplementedError('This family does not support RCStream') - - def rcstream_port(self, code): - """Return port for RCStream.""" - raise NotImplementedError('This family does not support RCStream') - def eventstreams_host(self, code): """Hostname for EventStreams.""" raise NotImplementedError('This family does not support EventStreams') @@ -1741,11 +1729,6 @@ """Return 'https' as the protocol.""" return 'https' - @deprecated('eventstreams_host', since='20170713') - def rcstream_host(self, code): - """DEPRECATED: use eventstreams_host instead.""" - return self.eventstreams_host(code) - def eventstreams_host(self, code): """Return 'https://stream.wikimedia.org' as the stream hostname.""" return 'https://stream.wikimedia.org' -- To view, visit https://gerrit.wikimedia.org/r/568982 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5b5c93cc6b1fe5a6b3554d160ac50522ab8e550d Gerrit-Change-Number: 568982 Gerrit-PatchSet: 1 Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: [bugfix] Remove isbn library of isbn.py in favour of stdnum
by jenkins-bot (Code Review) 30 Jan '20

30 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/565784 ) Change subject: [bugfix] Remove isbn library of isbn.py in favour of stdnum ...................................................................... [bugfix] Remove isbn library of isbn.py in favour of stdnum ISBN range of isbn.py is very outdated and updating it was declined in favour of using python-stdnum. isbn.py: - remove outdated ISBN ranges - remove ISBN classes and subclasses; this library part has been dropped in favour of python-stdnum package - remove getIsbn function which is not used anymore; any call to the intrinsic isbn library is replaced by a NotImplementedError exception - combine multiple exceptions in is_valid function - use suggest_help to show missing depedencies if isbn packages aren't installed cosmetic_changes.py: - don't attemp to use isbn.py methods; raise NotImplementesError instead if python-stdnum is not installed - call fix_ISBN if this library is available; otherwise skip it isbn_tests.py: - skip all tests if isbn libraries aren't installed - remove ISBN10/ISBN13 tests because these classes has been removed - use wikipedia family instead of test family because test family is deprecated Bug: T132919 Bug: T144288 Bug: T241141 Change-Id: Ib68d0c6898088425bad96c7ccfd4187dfd4bbdbf --- M pywikibot/cosmetic_changes.py M scripts/isbn.py M tests/isbn_tests.py 3 files changed, 55 insertions(+), 1,451 deletions(-) Approvals: Dvorapa: Looks good to me, approved jenkins-bot: Verified diff --git a/pywikibot/cosmetic_changes.py b/pywikibot/cosmetic_changes.py index 90847d1..051b06f 100755 --- a/pywikibot/cosmetic_changes.py +++ b/pywikibot/cosmetic_changes.py @@ -52,8 +52,8 @@ 'your_script_name_2'] """ # -# (C) xqt, 2009-2018 -# (C) Pywikibot team, 2006-2019 +# (C) xqt, 2009-2020 +# (C) Pywikibot team, 2006-2020 # # Distributed under the terms of the MIT license. # @@ -61,8 +61,6 @@ import re -from warnings import warn - try: import stdnum.isbn as stdnum_isbn except ImportError: @@ -170,48 +168,20 @@ def _format_isbn_match(match, strict=True): """Helper function to validate and format a single matched ISBN.""" - scripts_isbn = None - if not stdnum_isbn: - # For backwards compatibility, if stdnum.isbn is not available - # attempt loading scripts.isbn as an alternative implementation. - try: - import scripts.isbn as scripts_isbn - except ImportError: - raise NotImplementedError( - 'ISBN functionality not available. Install stdnum package.') - - warn('package stdnum.isbn not found; using scripts.isbn', - ImportWarning) + raise NotImplementedError( + 'ISBN functionality not available. Install stdnum package.') isbn = match.group('code') - if stdnum_isbn: - try: - stdnum_isbn.validate(isbn) - except stdnum_isbn.ValidationError as e: - if strict: - raise - pywikibot.log('ISBN "%s" validation error: %s' % (isbn, e)) - return isbn + try: + stdnum_isbn.validate(isbn) + except stdnum_isbn.ValidationError as e: + if strict: + raise + pywikibot.log('ISBN "%s" validation error: %s' % (isbn, e)) + return isbn - return stdnum_isbn.format(isbn) - else: - try: - scripts_isbn.is_valid(isbn) - except scripts_isbn.InvalidIsbnException as e: - if strict: - raise - pywikibot.log('ISBN "%s" validation error: %s' % (isbn, e)) - return isbn - - isbn = scripts_isbn.getIsbn(isbn) - try: - isbn.format() - except scripts_isbn.InvalidIsbnException as e: - if strict: - raise - pywikibot.log('ISBN "%s" validation error: %s' % (isbn, e)) - return isbn.code + return stdnum_isbn.format(isbn) def _reformat_ISBNs(text, strict=True): @@ -243,7 +213,7 @@ self.title = pageTitle self.ignore = ignore - self.common_methods = ( + self.common_methods = [ self.commonsfiledesc, self.fixSelfInterwiki, self.standardizePageFooter, @@ -265,9 +235,9 @@ self.fixTypo, self.fixArabicLetters, - # FIXME: T144288 - # self.fix_ISBN, - ) + ] + if stdnum_isbn: + self.common_methods.append(self.fix_ISBN) @classmethod def from_page(cls, page, diff, ignore): diff --git a/scripts/isbn.py b/scripts/isbn.py index 223aad1..429574c 100755 --- a/scripts/isbn.py +++ b/scripts/isbn.py @@ -47,6 +47,7 @@ import pywikibot from pywikibot import i18n, pagegenerators, textlib, Bot, WikidataBot +from pywikibot.tools import has_module try: import stdnum.isbn @@ -60,1298 +61,12 @@ '&params;': pagegenerators.parameterHelp, } -# Maps each group number to the list of its publisher number ranges. Source: -# https://web.archive.org/web/20090823122028/http://www.isbn-international.or… -ranges = { - '0': [ # English speaking area - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('900000', '949999'), - ('9500000', '9999999'), - ], - '1': [ # English speaking area - ('00', '09'), - ('100', '399'), - ('4000', '5499'), - ('55000', '86979'), - ('869800', '998999'), - ], - '2': [ # French speaking area - ('00', '19'), - ('200', '349'), - ('35000', '39999'), - ('400', '699'), - ('7000', '8399'), - ('84000', '89999'), - ('900000', '949999'), - ('9500000', '9999999'), - ], - '3': [ # German speaking area - ('00', '02'), - ('030', '033'), - ('0340', '0369'), - ('03700', '03999'), - ('04', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('900000', '949999'), - ('9500000', '9999999'), - ], - '4': [ # Japan - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('900000', '949999'), - ('9500000', '9999999'), - ], - '5': [ # Russian Federation - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('900000', '909999'), - ('91000', '91999'), - ('9200', '9299'), - ('93000', '94999'), - ('9500', '9799'), - ('98000', '98999'), - ('9900000', '9909999'), - ('9910', '9999'), - ], - '600': [ # Iran - ('00', '09'), - ('100', '499'), - ('5000', '8999'), - ('90000', '99999'), - ], - '601': [ # Kazakhstan - ('00', '19'), - ('200', '699'), - ('7000', '7999'), - ('80000', '84999'), - ('85', '99'), - ], - '602': [ # Indonesia - ('00', '19'), - ('200', '799'), - ('8000', '9499'), - ('95000', '99999'), - ], - '603': [ # Saudi Arabia - ('00', '04'), - ('500', '799'), - ('8000', '8999'), - ('90000', '99999'), - ], - '604': [ # Vietnam - ('0', '4'), - ('50', '89'), - ('900', '979'), - ('9800', '9999'), - ], - '605': [ # Turkey - ('00', '09'), - ('100', '399'), - ('4000', '5999'), - ('60000', '89999'), - ], - '7': [ # China, People's Republic - ('00', '09'), - ('100', '499'), - ('5000', '7999'), - ('80000', '89999'), - ('900000', '999999'), - ], - '80': [ # Czech Republic; Slovakia - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('900000', '999999'), - ], - '81': [ # India - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('900000', '999999'), - ], - '82': [ # Norway - ('00', '19'), - ('200', '699'), - ('7000', '8999'), - ('90000', '98999'), - ('990000', '999999'), - ], - '83': [ # Poland - ('00', '19'), - ('200', '599'), - ('60000', '69999'), - ('7000', '8499'), - ('85000', '89999'), - ('900000', '999999'), - ], - '84': [ # Spain - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('9000', '9199'), - ('920000', '923999'), - ('92400', '92999'), - ('930000', '949999'), - ('95000', '96999'), - ('9700', '9999'), - ], - '85': [ # Brazil - ('00', '19'), - ('200', '599'), - ('60000', '69999'), - ('7000', '8499'), - ('85000', '89999'), - ('900000', '979999'), - ('98000', '99999'), - ], - '86': [ # Serbia and Montenegro - ('00', '29'), - ('300', '599'), - ('6000', '7999'), - ('80000', '89999'), - ('900000', '999999'), - ], - '87': [ # Denmark - ('00', '29'), - ('400', '649'), - ('7000', '7999'), - ('85000', '94999'), - ('970000', '999999'), - ], - '88': [ # Italian speaking area - ('00', '19'), - ('200', '599'), - ('6000', '8499'), - ('85000', '89999'), - ('900000', '949999'), - ('95000', '99999'), - ], - '89': [ # Korea - ('00', '24'), - ('250', '549'), - ('5500', '8499'), - ('85000', '94999'), - ('950000', '999999'), - ], - '90': [ # Netherlands, Belgium (Flemish) - ('00', '19'), - ('200', '499'), - ('5000', '6999'), - ('70000', '79999'), - ('800000', '849999'), - ('8500', '8999'), - ('900000', '909999'), - ('940000', '949999'), - ], - '91': [ # Sweden - ('0', '1'), - ('20', '49'), - ('500', '649'), - ('7000', '7999'), - ('85000', '94999'), - ('970000', '999999'), - ], - # International Publishers (Unesco, EU), European Community Organizations - '92': [ - ('0', '5'), - ('60', '79'), - ('800', '899'), - ('9000', '9499'), - ('95000', '98999'), - ('990000', '999999'), - ], - '93': [ # India - no ranges fixed yet - ], - '950': [ # Argentina - ('00', '49'), - ('500', '899'), - ('9000', '9899'), - ('99000', '99999'), - ], - '951': [ # Finland - ('0', '1'), - ('20', '54'), - ('550', '889'), - ('8900', '9499'), - ('95000', '99999'), - ], - '952': [ # Finland - ('00', '19'), - ('200', '499'), - ('5000', '5999'), - ('60', '65'), - ('6600', '6699'), - ('67000', '69999'), - ('7000', '7999'), - ('80', '94'), - ('9500', '9899'), - ('99000', '99999'), - ], - '953': [ # Croatia - ('0', '0'), - ('10', '14'), - ('150', '549'), - ('55000', '59999'), - ('6000', '9499'), - ('95000', '99999'), - ], - '954': [ # Bulgaria - ('00', '29'), - ('300', '799'), - ('8000', '8999'), - ('90000', '92999'), - ('9300', '9999'), - ], - '955': [ # Sri Lanka - ('0', '0'), - ('1000', '1999'), - ('20', '54'), - ('550', '799'), - ('8000', '9499'), - ('95000', '99999'), - ], - '956': [ # Chile - ('00', '19'), - ('200', '699'), - ('7000', '9999'), - ], - '957': [ # Taiwan, China - ('00', '02'), - ('0300', '0499'), - ('05', '19'), - ('2000', '2099'), - ('21', '27'), - ('28000', '30999'), - ('31', '43'), - ('440', '819'), - ('8200', '9699'), - ('97000', '99999'), - ], - '958': [ # Colombia - ('00', '59'), - ('600', '799'), - ('8000', '9499'), - ('95000', '99999'), - ], - '959': [ # Cuba - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ], - '960': [ # Greece - ('00', '19'), - ('200', '659'), - ('6600', '6899'), - ('690', '699'), - ('7000', '8499'), - ('85000', '99999'), - ], - '961': [ # Slovenia - ('00', '19'), - ('200', '599'), - ('6000', '8999'), - ('90000', '94999'), - ], - '962': [ # Hong Kong - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '86999'), - ('8700', '8999'), - ('900', '999'), - ], - '963': [ # Hungary - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('9000', '9999'), - ], - '964': [ # Iran - ('00', '14'), - ('150', '249'), - ('2500', '2999'), - ('300', '549'), - ('5500', '8999'), - ('90000', '96999'), - ('970', '989'), - ('9900', '9999'), - ], - '965': [ # Israel - ('00', '19'), - ('200', '599'), - ('7000', '7999'), - ('90000', '99999'), - ], - '966': [ # Ukraine - ('00', '19'), - ('2000', '2999'), - ('300', '699'), - ('7000', '8999'), - ('90000', '99999'), - ], - '967': [ # Malaysia - ('00', '29'), - ('300', '499'), - ('5000', '5999'), - ('60', '89'), - ('900', '989'), - ('9900', '9989'), - ('99900', '99999'), - ], - '968': [ # Mexico - ('01', '39'), - ('400', '499'), - ('5000', '7999'), - ('800', '899'), - ('9000', '9999'), - ], - '969': [ # Pakistan - ('0', '1'), - ('20', '39'), - ('400', '799'), - ('8000', '9999'), - ], - '970': [ # Mexico - ('01', '59'), - ('600', '899'), - ('9000', '9099'), - ('91000', '96999'), - ('9700', '9999'), - ], - '971': [ # Philippines? - ('000', '019'), - ('02', '02'), - ('0300', '0599'), - ('06', '09'), - ('10', '49'), - ('500', '849'), - ('8500', '9099'), - ('91000', '99999'), - ], - '972': [ # Portugal - ('0', '1'), - ('20', '54'), - ('550', '799'), - ('8000', '9499'), - ('95000', '99999'), - ], - '973': [ # Romania - ('0', '0'), - ('100', '169'), - ('1700', '1999'), - ('20', '54'), - ('550', '759'), - ('7600', '8499'), - ('85000', '88999'), - ('8900', '9499'), - ('95000', '99999'), - ], - '974': [ # Thailand - ('00', '19'), - ('200', '699'), - ('7000', '8499'), - ('85000', '89999'), - ('90000', '94999'), - ('9500', '9999'), - ], - '975': [ # Turkey - ('00000', '00999'), - ('01', '24'), - ('250', '599'), - ('6000', '9199'), - ('92000', '98999'), - ('990', '999'), - ], - '976': [ # Caribbean Community - ('0', '3'), - ('40', '59'), - ('600', '799'), - ('8000', '9499'), - ('95000', '99999'), - ], - '977': [ # Egypt - ('00', '19'), - ('200', '499'), - ('5000', '6999'), - ('700', '999'), - ], - '978': [ # Nigeria - ('000', '199'), - ('2000', '2999'), - ('30000', '79999'), - ('8000', '8999'), - ('900', '999'), - ], - '979': [ # Indonesia - ('000', '099'), - ('1000', '1499'), - ('15000', '19999'), - ('20', '29'), - ('3000', '3999'), - ('400', '799'), - ('8000', '9499'), - ('95000', '99999'), - ], - '980': [ # Venezuela - ('00', '19'), - ('200', '599'), - ('6000', '9999'), - ], - '981': [ # Singapore - ('00', '19'), - ('200', '299'), - ('3000', '9999'), - ], - '982': [ # South Pacific - ('00', '09'), - ('100', '699'), - ('70', '89'), - ('9000', '9999'), - ], - '983': [ # Malaysia - ('00', '01'), - ('020', '199'), - ('2000', '3999'), - ('40000', '44999'), - ('45', '49'), - ('50', '79'), - ('800', '899'), - ('9000', '9899'), - ('99000', '99999'), - ], - '984': [ # Bangladesh - ('00', '39'), - ('400', '799'), - ('8000', '8999'), - ('90000', '99999'), - ], - '985': [ # Belarus - ('00', '39'), - ('400', '599'), - ('6000', '8999'), - ('90000', '99999'), - ], - '986': [ # Taiwan, China - ('00', '11'), - ('120', '559'), - ('5600', '7999'), - ('80000', '99999'), - ], - '987': [ # Argentina - ('00', '09'), - ('1000', '1999'), - ('20000', '29999'), - ('30', '49'), - ('500', '899'), - ('9000', '9499'), - ('95000', '99999'), - ], - '988': [ # Hongkong - ('00', '16'), - ('17000', '19999'), - ('200', '799'), - ('8000', '9699'), - ('97000', '99999'), - ], - '989': [ # Portugal - ('0', '1'), - ('20', '54'), - ('550', '799'), - ('8000', '9499'), - ('95000', '99999'), - ], - '9937': [ # Nepal - ('0', '2'), - ('30', '49'), - ('500', '799'), - ('8000', '9999'), - ], - '9938': [ # Tunisia - ('00', '79'), - ('800', '949'), - ('9500', '9999'), - ], - '9939': [ # Armenia - ('0', '4'), - ('50', '79'), - ('800', '899'), - ('9000', '9999'), - ], - '9940': [ # Montenegro - ('0', '1'), - ('20', '49'), - ('500', '899'), - ('9000', '9999'), - ], - '9941': [ # Georgia - ('0', '0'), - ('10', '39'), - ('400', '899'), - ('9000', '9999'), - ], - '9942': [ # Ecuador - ('00', '89'), - ('900', '994'), - ('9950', '9999'), - ], - '9943': [ # Uzbekistan - ('00', '29'), - ('300', '399'), - ('4000', '9999'), - ], - '9944': [ # Turkey - ('0', '2'), - ('300', '499'), - ('5000', '5999'), - ('60', '89'), - ('900', '999'), - ], - '9945': [ # Dominican Republic - ('00', '00'), - ('010', '079'), - ('08', '39'), - ('400', '569'), - ('57', '57'), - ('580', '849'), - ('8500', '9999'), - ], - '9946': [ # Korea, P.D.R. - ('0', '1'), - ('20', '39'), - ('400', '899'), - ('9000', '9999'), - ], - '9947': [ # Algeria - ('0', '1'), - ('20', '79'), - ('800', '999'), - ], - '9948': [ # United Arab Emirates - ('00', '39'), - ('400', '849'), - ('8500', '9999'), - ], - '9949': [ # Estonia - ('0', '0'), - ('10', '39'), - ('400', '899'), - ('9000', '9999'), - ], - '9950': [ # Palestine - ('00', '29'), - ('300', '840'), - ('8500', '9999'), - ], - '9951': [ # Kosova - ('00', '39'), - ('400', '849'), - ('8500', '9999'), - ], - '9952': [ # Azerbaijan - ('0', '1'), - ('20', '39'), - ('400', '799'), - ('8000', '9999'), - ], - '9953': [ # Lebanon - ('0', '0'), - ('10', '39'), - ('400', '599'), - ('60', '89'), - ('9000', '9999'), - ], - '9954': [ # Morocco - ('0', '1'), - ('20', '39'), - ('400', '799'), - ('8000', '9999'), - ], - '9955': [ # Lithuania - ('00', '39'), - ('400', '929'), - ('9300', '9999'), - ], - '9956': [ # Cameroon - ('0', '0'), - ('10', '39'), - ('400', '899'), - ('9000', '9999'), - ], - '9957': [ # Jordan - ('00', '39'), - ('400', '699'), - ('70', '84'), - ('8500', '9999'), - ], - '9958': [ # Bosnia and Herzegovina - ('0', '0'), - ('10', '49'), - ('500', '899'), - ('9000', '9999'), - ], - '9959': [ # Libya - ('0', '1'), - ('20', '79'), - ('800', '949'), - ('9500', '9999'), - ], - '9960': [ # Saudi Arabia - ('00', '59'), - ('600', '899'), - ('9000', '9999'), - ], - '9961': [ # Algeria - ('0', '2'), - ('30', '69'), - ('700', '949'), - ('9500', '9999'), - ], - '9962': [ # Panama - ('00', '54'), - ('5500', '5599'), - ('56', '59'), - ('600', '849'), - ('8500', '9999'), - ], - '9963': [ # Cyprus - ('0', '2'), - ('30', '54'), - ('550', '749'), - ('7500', '9999'), - ], - '9964': [ # Ghana - ('0', '6'), - ('70', '94'), - ('950', '999'), - ], - '9965': [ # Kazakhstan - ('00', '39'), - ('400', '899'), - ('9000', '9999'), - ], - '9966': [ # Kenya - ('00', '69'), - ('7000', '7499'), - ('750', '959'), - ('9600', '9999'), - ], - '9967': [ # Kyrgyzstan - ('00', '39'), - ('400', '899'), - ('9000', '9999'), - ], - '9968': [ # Costa Rica - ('00', '49'), - ('500', '939'), - ('9400', '9999'), - ], - '9970': [ # Uganda - ('00', '39'), - ('400', '899'), - ('9000', '9999'), - ], - '9971': [ # Singapore - ('0', '5'), - ('60', '89'), - ('900', '989'), - ('9900', '9999'), - ], - '9972': [ # Peru - ('00', '09'), - ('1', '1'), - ('200', '249'), - ('2500', '2999'), - ('30', '59'), - ('600', '899'), - ('9000', '9999'), - ], - '9973': [ # Tunisia - ('0', '05'), - ('060', '089'), - ('0900', '0999'), - ('10', '69'), - ('700', '969'), - ('9700', '9999'), - ], - '9974': [ # Uruguay - ('0', '2'), - ('30', '54'), - ('550', '749'), - ('7500', '9499'), - ('95', '99'), - ], - '9975': [ # Moldova - ('0', '0'), - ('100', '399'), - ('4000', '4499'), - ('45', '89'), - ('900', '949'), - ('9500', '9999'), - ], - '9976': [ # Tanzania - ('0', '5'), - ('60', '89'), - ('900', '989'), - ('9990', '9999'), - ], - '9977': [ # Costa Rica - ('00', '89'), - ('900', '989'), - ('9900', '9999'), - ], - '9978': [ # Ecuador - ('00', '29'), - ('300', '399'), - ('40', '94'), - ('950', '989'), - ('9900', '9999'), - ], - '9979': [ # Iceland - ('0', '4'), - ('50', '64'), - ('650', '659'), - ('66', '75'), - ('760', '899'), - ('9000', '9999'), - ], - '9980': [ # Papua New Guinea - ('0', '3'), - ('40', '89'), - ('900', '989'), - ('9900', '9999'), - ], - '9981': [ # Morocco - ('00', '09'), - ('100', '159'), - ('1600', '1999'), - ('20', '79'), - ('800', '949'), - ('9500', '9999'), - ], - '9982': [ # Zambia - ('00', '79'), - ('800', '989'), - ('9900', '9999'), - ], - '9983': [ # Gambia - ('80', '94'), - ('950', '989'), - ('9900', '9999'), - ], - '9984': [ # Latvia - ('00', '49'), - ('500', '899'), - ('9000', '9999'), - ], - '9985': [ # Estonia - ('0', '4'), - ('50', '79'), - ('800', '899'), - ('9000', '9999'), - ], - '9986': [ # Lithuania - ('00', '39'), - ('400', '899'), - ('9000', '9399'), - ('940', '969'), - ('97', '99'), - ], - '9987': [ # Tanzania - ('00', '39'), - ('400', '879'), - ('8800', '9999'), - ], - '9988': [ # Ghana - ('0', '2'), - ('30', '54'), - ('550', '749'), - ('7500', '9999'), - ], - '9989': [ # Macedonia - ('0', '0'), - ('100', '199'), - ('2000', '2999'), - ('30', '59'), - ('600', '949'), - ('9500', '9999'), - ], - '99901': [ # Bahrain - ('00', '49'), - ('500', '799'), - ('80', '99'), - ], - '99902': [ # Gabon - no ranges fixed yet - ], - '99903': [ # Mauritius - ('0', '1'), - ('20', '89'), - ('900', '999'), - ], - '99904': [ # Netherlands Antilles; Aruba, Neth. Ant - ('0', '5'), - ('60', '89'), - ('900', '999'), - ], - '99905': [ # Bolivia - ('0', '3'), - ('40', '79'), - ('800', '999'), - ], - '99906': [ # Kuwait - ('0', '2'), - ('30', '59'), - ('600', '699'), - ('70', '89'), - ('9', '9'), - ], - '99908': [ # Malawi - ('0', '0'), - ('10', '89'), - ('900', '999'), - ], - '99909': [ # Malta - ('0', '3'), - ('40', '94'), - ('950', '999'), - ], - '99910': [ # Sierra Leone - ('0', '2'), - ('30', '89'), - ('900', '999'), - ], - '99911': [ # Lesotho - ('00', '59'), - ('600', '999'), - ], - '99912': [ # Botswana - ('0', '3'), - ('400', '599'), - ('60', '89'), - ('900', '999'), - ], - '99913': [ # Andorra - ('0', '2'), - ('30', '35'), - ('600', '604'), - ], - '99914': [ # Suriname - ('0', '4'), - ('50', '89'), - ('900', '949'), - ], - '99915': [ # Maldives - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99916': [ # Namibia - ('0', '2'), - ('30', '69'), - ('700', '999'), - ], - '99917': [ # Brunei Darussalam - ('0', '2'), - ('30', '89'), - ('900', '999'), - ], - '99918': [ # Faroe Islands - ('0', '3'), - ('40', '79'), - ('800', '999'), - ], - '99919': [ # Benin - ('0', '2'), - ('40', '69'), - ('900', '999'), - ], - '99920': [ # Andorra - ('0', '4'), - ('50', '89'), - ('900', '999'), - ], - '99921': [ # Qatar - ('0', '1'), - ('20', '69'), - ('700', '799'), - ('8', '8'), - ('90', '99'), - ], - '99922': [ # Guatemala - ('0', '3'), - ('40', '69'), - ('700', '999'), - ], - '99923': [ # El Salvador - ('0', '1'), - ('20', '79'), - ('800', '999'), - ], - '99924': [ # Nicaragua - ('0', '2'), - ('30', '79'), - ('800', '999'), - ], - '99925': [ # Paraguay - ('0', '3'), - ('40', '79'), - ('800', '999'), - ], - '99926': [ # Honduras - ('0', '0'), - ('10', '59'), - ('600', '999'), - ], - '99927': [ # Albania - ('0', '2'), - ('30', '59'), - ('600', '999'), - ], - '99928': [ # Georgia - ('0', '0'), - ('10', '79'), - ('800', '999'), - ], - '99929': [ # Mongolia - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99930': [ # Armenia - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99931': [ # Seychelles - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99932': [ # Malta - ('0', '0'), - ('10', '59'), - ('600', '699'), - ('7', '7'), - ('80', '99'), - ], - '99933': [ # Nepal - ('0', '2'), - ('30', '59'), - ('600', '999'), - ], - '99934': [ # Dominican Republic - ('0', '1'), - ('20', '79'), - ('800', '999'), - ], - '99935': [ # Haiti - ('0', '2'), - ('7', '8'), - ('30', '59'), - ('600', '699'), - ('90', '99'), - ], - '99936': [ # Bhutan - ('0', '0'), - ('10', '59'), - ('600', '999'), - ], - '99937': [ # Macau - ('0', '1'), - ('20', '59'), - ('600', '999'), - ], - '99938': [ # Srpska - ('0', '1'), - ('20', '59'), - ('600', '899'), - ('90', '99'), - ], - '99939': [ # Guatemala - ('0', '5'), - ('60', '89'), - ('900', '999'), - ], - '99940': [ # Georgia - ('0', '0'), - ('10', '69'), - ('700', '999'), - ], - '99941': [ # Armenia - ('0', '2'), - ('30', '79'), - ('800', '999'), - ], - '99942': [ # Sudan - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99943': [ # Alsbania - ('0', '2'), - ('30', '59'), - ('600', '999'), - ], - '99944': [ # Ethiopia - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99945': [ # Namibia - ('0', '5'), - ('60', '89'), - ('900', '999'), - ], - '99946': [ # Nepal - ('0', '2'), - ('30', '59'), - ('600', '999'), - ], - '99947': [ # Tajikistan - ('0', '2'), - ('30', '69'), - ('700', '999'), - ], - '99948': [ # Eritrea - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99949': [ # Mauritius - ('0', '1'), - ('20', '89'), - ('900', '999'), - ], - '99950': [ # Cambodia - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99951': [ # Congo - no ranges fixed yet - ], - '99952': [ # Mali - ('0', '4'), - ('50', '79'), - ('800', '999'), - ], - '99953': [ # Paraguay - ('0', '2'), - ('30', '79'), - ('800', '999'), - ], - '99954': [ # Bolivia - ('0', '2'), - ('30', '69'), - ('700', '999'), - ], - '99955': [ # Srpska - ('0', '1'), - ('20', '59'), - ('600', '899'), - ('90', '99'), - ], - '99956': [ # Albania - ('00', '59'), - ('600', '999'), - ], -} - class InvalidIsbnException(pywikibot.Error): """Invalid ISBN.""" -class ISBN(object): - - """Abstract superclass.""" - - def format(self): - """Put hyphens into this ISBN number.""" - result = '' - rest = '' - for digit in self.digits(): - rest += str(digit) - # Determine the prefix (if any) - for prefix in self.possiblePrefixes(): - if rest.startswith(prefix): - result += prefix + '-' - rest = rest[len(prefix):] - break - - # Determine the group - for groupNumber in ranges.keys(): - if rest.startswith(groupNumber): - result += groupNumber + '-' - rest = rest[len(groupNumber):] - publisherRanges = ranges[groupNumber] - break - else: - raise InvalidIsbnException('ISBN {0}: group number unknown.' - .format(self.code)) - - # Determine the publisher - for (start, end) in publisherRanges: - length = len(start) # NOTE: start and end always have equal length - if rest[:length] >= start and rest[:length] <= end: - result += rest[:length] + '-' - rest = rest[length:] - break - else: - raise InvalidIsbnException('ISBN {0}: publisher number unknown.' - .format(self.code)) - - # The rest is the item number and the 1-digit checksum. - result += rest[:-1] + '-' + rest[-1] - self.code = result - - -class ISBN13(ISBN): - - """ISBN 13.""" - - def __init__(self, code, checksumMissing=False): - """Initializer.""" - self.code = code - if checksumMissing: - self.code += str(self.calculateChecksum()) - self.checkValidity() - - def possiblePrefixes(self): - """Return possible prefixes.""" - return ['978', '979'] - - def digits(self): - """Return a list of the digits in the ISBN code.""" - result = [] - for c in self.code: - if c.isdigit(): - result.append(int(c)) - elif c != '-': - raise InvalidIsbnException( - 'The ISBN {0} contains invalid characters.' - .format(self.code)) - return result - - def checkValidity(self): - """Check validity of ISBN.""" - if len(self.digits()) != 13: - raise InvalidIsbnException('The ISBN {0} is not 13 digits long.' - .format(self.code)) - if self.calculateChecksum() != self.digits()[-1]: - raise InvalidIsbnException('The ISBN checksum of {0} is incorrect.' - .format(self.code)) - - def calculateChecksum(self): - """ - Calculate checksum. - - See https://en.wikipedia.org/wiki/ISBN#Check_digit_in_ISBN_13 - """ - sum = 0 - for i in range(0, 13 - 1, 2): - sum += self.digits()[i] - for i in range(1, 13 - 1, 2): - sum += 3 * self.digits()[i] - return (10 - (sum % 10)) % 10 - - -class ISBN10(ISBN): - - """ISBN 10.""" - - def __init__(self, code): - """Initializer.""" - self.code = code - self.checkValidity() - - def possiblePrefixes(self): - """Return possible prefixes.""" - return [] - - def digits(self): - """Return a list of the digits and Xs in the ISBN code.""" - result = [] - for c in self.code: - if c.isdigit() or c in 'Xx': - result.append(c) - elif c != '-': - raise InvalidIsbnException( - 'The ISBN {0} contains invalid characters.' - .format(self.code)) - return result - - def checkChecksum(self): - """Raise an InvalidIsbnException if the ISBN checksum is incorrect.""" - # See https://en.wikipedia.org/wiki/ISBN#Check_digit_in_ISBN_10 - sum = 0 - for i in range(0, 9): - sum += (i + 1) * int(self.digits()[i]) - checksum = sum % 11 - lastDigit = self.digits()[-1] - if not (checksum == 10 and lastDigit in 'Xx' - or lastDigit.isdigit() and checksum == int(lastDigit)): - raise InvalidIsbnException('The ISBN checksum of {0} is incorrect.' - .format(self.code)) - - def checkValidity(self): - """Check validity of ISBN.""" - if len(self.digits()) != 10: - raise InvalidIsbnException('The ISBN {0} is not 10 digits long.' - .format(self.code)) - if 'X' in self.digits()[:-1] or 'x' in self.digits()[:-1]: - raise InvalidIsbnException( - 'ISBN {0}: X is only allowed at the end of the ISBN.' - .format(self.code)) - self.checkChecksum() - - def toISBN13(self): - """ - Create a 13-digit ISBN from this 10-digit ISBN. - - Adds the GS1 prefix '978' and recalculates the checksum. - The hyphenation structure is taken from the format of the original - ISBN number. - - @rtype: L{ISBN13} - """ - code = '978-' + self.code[:-1] - return ISBN13(code, checksumMissing=True) - - def format(self): - """Format ISBN number.""" - # load overridden superclass method - ISBN.format(self) - # capitalize checksum - if self.code[-1] == 'x': - self.code = self.code[:-1] + 'X' - - -def getIsbn(code): - """Return an ISBN object for the code.""" - try: - i = ISBN13(code) - except InvalidIsbnException as e13: - try: - i = ISBN10(code) - except InvalidIsbnException as e10: - raise InvalidIsbnException('ISBN-13: {0} / ISBN-10: {1}' - .format(e13, e10)) - return i - - def is_valid(isbn): """Check whether an ISBN 10 or 13 is valid.""" # isbnlib marks any ISBN10 with lowercase 'X' as invalid @@ -1363,11 +78,9 @@ else: try: stdnum.isbn.validate(isbn) - except stdnum.isbn.InvalidFormat as e: - raise InvalidIsbnException(str(e)) - except stdnum.isbn.InvalidChecksum as e: - raise InvalidIsbnException(str(e)) - except stdnum.isbn.InvalidLength as e: + except (stdnum.isbn.InvalidFormat, + stdnum.isbn.InvalidChecksum, + stdnum.isbn.InvalidLength) as e: raise InvalidIsbnException(str(e)) return True @@ -1380,8 +93,8 @@ raise InvalidIsbnException('Invalid ISBN found') return True - getIsbn(isbn) - return True + raise NotImplementedError( + 'ISBN functionality not available. Install stdnum package.') def _hyphenateIsbnNumber(match): @@ -1398,12 +111,7 @@ except NameError: pass else: - i = stdnum.isbn.format(isbn) - return i - - i = getIsbn(isbn) - i.format() - return i.code + return stdnum.isbn.format(isbn) hyphenateIsbnNumbers = partial(textlib.reformat_ISBNs, @@ -1415,26 +123,23 @@ isbn = match.group('code') isbn = isbn.upper() try: + is_valid(isbn) + except InvalidIsbnException: + # don't change + return isbn + + try: stdnum.isbn except NameError: pass else: - try: - is_valid(isbn) - except InvalidIsbnException: - return isbn - i = stdnum.isbn.to_isbn13(isbn) - return i + return stdnum.isbn.to_isbn13(isbn) try: isbnlib except NameError: pass else: - try: - is_valid(isbn) - except InvalidIsbnException: - return isbn # remove hyphenation, otherwise isbnlib.to_isbn13() returns None i = isbnlib.canonical(isbn) if i == isbn: @@ -1445,18 +150,6 @@ i13h = hyphenateIsbnNumbers('ISBN ' + i13) return i13h[5:] - try: - is_valid(isbn) - except InvalidIsbnException: - # don't change - return isbn - i1x = getIsbn(isbn) - if not isinstance(i1x, ISBN13): - i13 = i1x.toISBN13() - else: - i13 = i1x - return i13.code - def convertIsbn10toIsbn13(text): """Helper function to convert ISBN 10 to ISBN 13.""" @@ -1647,14 +340,20 @@ genFactory.handleArg(arg) gen = genFactory.getCombinedGenerator(preload=True) - if gen: - if use_wikibase: - bot = IsbnWikibaseBot(gen, **options) - else: - bot = IsbnBot(gen, **options) - bot.run() + + deps = [] + if not (has_module('stdnum', version='1.13') + or has_module('isbnlib', version='3.9.10')): + deps = ["'python-stdnum >= 1.13' or 'isbnlib >= 3.9.10'"] + if pywikibot.bot.suggest_help(missing_generator=not gen, + missing_dependencies=deps): + return + + if use_wikibase: + bot = IsbnWikibaseBot(gen, **options) else: - pywikibot.bot.suggest_help(missing_generator=True) + bot = IsbnBot(gen, **options) + bot.run() if __name__ == '__main__': diff --git a/tests/isbn_tests.py b/tests/isbn_tests.py index ee74120..361f40f 100644 --- a/tests/isbn_tests.py +++ b/tests/isbn_tests.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- """Tests for isbn script.""" # -# (C) Pywikibot team, 2014-2019 +# (C) Pywikibot team, 2014-2020 # # Distributed under the terms of the MIT license. # @@ -16,14 +16,14 @@ from pywikibot import Bot, Claim, ItemPage from pywikibot.cosmetic_changes import CosmeticChangesToolkit, CANCEL_MATCH +from pywikibot.tools import has_module from scripts.isbn import ( - ISBN10, ISBN13, InvalidIsbnException as IsbnExc, - getIsbn, hyphenateIsbnNumbers, convertIsbn10toIsbn13, + InvalidIsbnException as IsbnExc, + hyphenateIsbnNumbers, convertIsbn10toIsbn13, main ) -from tests import patch, Mock from tests.aspects import ( unittest, TestCase, DefaultDrySiteTestCase, WikibaseTestCase, ScriptMainTestCase, @@ -36,8 +36,6 @@ AnyIsbnValidationException = IsbnExc -# Suppress ImportWarning: package stdnum.isbn not found; using scripts.isbn -@patch('pywikibot.cosmetic_changes.warn', Mock()) class TestCosmeticChangesISBN(DefaultDrySiteTestCase): """Test CosmeticChanges ISBN fix.""" @@ -52,14 +50,6 @@ text = cc.fix_ISBN(' ISBN 9780975229804 ') self.assertEqual(text, ' ISBN 978-0-9752298-0-4 ') - @unittest.expectedFailure # T144288 - def test_valid_isbn_failing(self): - """Test ISBN. - - This test fails with current library parts. - """ - cc = CosmeticChangesToolkit(self.site, namespace=0) - text = cc.fix_ISBN(' ISBN 9783955390631 ') self.assertEqual(text, ' ISBN 978-3-95539-063-1 ') @@ -98,60 +88,8 @@ net = False - def test_isbn10(self): - """Test ISBN10.""" - # Test general features - isbn = ISBN10('097522980x') - isbn.format() - self.assertEqual(isbn.code, '0-9752298-0-X') - self.assertEqual(isbn.digits(), - ['0', '9', '7', '5', '2', '2', '9', '8', '0', 'X']) - - # Converting to ISBN13 - isbn13 = isbn.toISBN13() - self.assertEqual(isbn13.code, '978-0-9752298-0-4') - - # Errors - self.assertRaises(IsbnExc, ISBN10, '0975229LOL') # Invalid characters - self.assertRaises(IsbnExc, ISBN10, '0975229801') # Invalid checksum - self.assertRaises(IsbnExc, ISBN10, '09752298') # Invalid length - self.assertRaises(IsbnExc, ISBN10, '09752X9801') # X in the middle - - def test_isbn13_978(self): - """General test ISBN13 features with prefix 978.""" - isbn = ISBN13('9783161484100') - isbn.format() - self.assertEqual(isbn.code, '978-3-16-148410-0') - self.assertEqual(isbn.digits(), - [9, 7, 8, 3, 1, 6, 1, 4, 8, 4, 1, 0, 0]) - - isbn = ISBN13('978809027341', checksumMissing=True) - self.assertEqual(isbn.code, '9788090273412') - - # Errors - self.assertRaises(IsbnExc, ISBN13, '9783161484LOL') # Invalid chars - self.assertRaises(IsbnExc, ISBN13, '9783161484105') # Invalid checksum - self.assertRaises(IsbnExc, ISBN13, '9783161484') # Invalid length - - @unittest.expectedFailure # T144288 - def test_isbn13_979(self): - """Test ISBN13 with prefix 979.""" - isbn = ISBN13('9791091447089') - isbn.format() - self.assertEqual(isbn.code, '979-10-91447-08-9') - self.assertEqual(isbn.digits(), - [9, 7, 9, 1, 0, 9, 1, 4, 4, 7, 0, 8, 9]) - def test_general(self): """Test things that apply both to ISBN10 and ISBN13.""" - # getIsbn - self.assertIsInstance(getIsbn('097522980x'), ISBN10) - self.assertIsInstance(getIsbn('9783161484100'), ISBN13) - self.assertRaisesRegex(IsbnExc, - 'ISBN-13: The ISBN 097522 is not 13 digits ' - 'long. / ISBN-10: The ISBN 097522 is not 10 ' - 'digits long.', getIsbn, '097522') - # hyphenateIsbnNumbers self.assertEqual(hyphenateIsbnNumbers('ISBN 097522980x'), 'ISBN 0-9752298-0-X') @@ -169,16 +107,6 @@ 'ISBN 978-0-7869-3669-4' ) - # Errors - isbn = ISBN10('9912098056') - self.assertRaisesRegex(IsbnExc, - 'ISBN 9912098056: group number unknown.', - isbn.format) - isbn = ISBN10('9095012042') - self.assertRaisesRegex(IsbnExc, - 'ISBN 9095012042: publisher number unknown.', - isbn.format) - @unittest.expectedFailure # T144288 def test_general_failing(self): """Test things that apply both to ISBN10 and ISBN13. @@ -197,7 +125,7 @@ """Test isbnbot with non-write patching (if the testpage exists).""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -301,5 +229,12 @@ pass +def setUpModule(): # noqa: N802 + """Skip tests if isbn libraries are missing.""" + if not (has_module('stdnum', version='1.13') + or has_module('isbnlib', version='3.9.10')): + raise unittest.SkipTest('neither python-stdlib nor isbnlib available.') + + if __name__ == '__main__': # pragma: no cover unittest.main() -- To view, visit https://gerrit.wikimedia.org/r/565784 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib68d0c6898088425bad96c7ccfd4187dfd4bbdbf Gerrit-Change-Number: 565784 Gerrit-PatchSet: 4 Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: D3r1ck <alangiderick(a)gmail.com> Gerrit-Reviewer: D3r1ck01 <xsavitar.wiki(a)aol.com> Gerrit-Reviewer: Dalba <dalba.wiki(a)gmail.com> Gerrit-Reviewer: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: Framawiki <framawiki(a)tools.wmflabs.org> Gerrit-Reviewer: Huji <huji.huji(a)gmail.com> Gerrit-Reviewer: JJMC89 <JJMC89.Wikimedia(a)gmail.com> Gerrit-Reviewer: Ladsgroup <Ladsgroup(a)gmail.com> Gerrit-Reviewer: Legoktm <legoktm(a)member.fsf.org> Gerrit-Reviewer: Matěj Suchánek <matejsuchanek97(a)gmail.com> Gerrit-Reviewer: Mpaa <mpaa.wiki(a)gmail.com> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: Zhuyifei1999 <zhuyifei1999(a)gmail.com> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: [doc] update tests/README.rst
by jenkins-bot (Code Review) 30 Jan '20

30 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/567290 ) Change subject: [doc] update tests/README.rst ...................................................................... [doc] update tests/README.rst - add a hint that -lang and -family option can be used with pwb.py - some minor changes Change-Id: I1711c8b218f97397937e56d909d1135ff1a0fd24 --- M tests/README.rst 1 file changed, 9 insertions(+), 6 deletions(-) Approvals: Dvorapa: Looks good to me, approved jenkins-bot: Verified diff --git a/tests/README.rst b/tests/README.rst index b2e8500..56984ec 100644 --- a/tests/README.rst +++ b/tests/README.rst @@ -70,7 +70,9 @@ Specific tests -------------- -Individual test components can be run using unittest, nosetests, or pwb +Individual test components can be run using unittest, nosetests, or pwb. +With -lang and -family options pwb can be used to specify a site. + unittest ~~~~~~~~ @@ -104,6 +106,7 @@ python pwb.py tests/api_tests -v python pwb.py tests/site_tests -v python pwb.py tests/api_tests -v TestParamInfo.test_init + python pwb.py -lang:de -family:wikipedia tests/page_tests -v TestPageObject env ~~~ @@ -123,10 +126,10 @@ https://github.com/wikimedia/pywikibot , and travis tests are run and published at https://travis-ci.org/wikimedia/pywikibot/builds . These tests use the Wikimedia global (SUL) account 'Pywikibot-test', which has a password -securely stored in .travis.yml . See section env:global:secure. +securely stored in .travis.yml file. See section env:global:secure. Anyone can run these tests on travis-ci.org using their own github account, with -code changes that have not been merged into the main repository. To do this: +code changes that have not been merged into the main repository. To do this: 1. create a github and travis-ci account 2. fork the main github repository https://github.com/wikimedia/pywikibot @@ -135,7 +138,7 @@ 5. watch the build at https://travis-ci.org/<username>/pywikibot/builds Only travis-ci builds from the main repository can access the password for the -Wikimedia account 'Pywikibot-test'. All tests which require a logged in user +Wikimedia account 'Pywikibot-test'. All tests which require a logged in user are skipped if the travis-ci build environment does not have a password. To enable 'user' tests on travis-ci builds for a different repository, add @@ -166,7 +169,7 @@ After changes are published into a github repository, tests may be run on a Microsoft Windows box provided by ci.appveyor.com according to the -configuration in .appveyor.yml . To do this: +configuration in .appveyor.yml file. To do this: 1. create a github and appveyor account 2. fork the main github repository @@ -207,7 +210,7 @@ ===================== There are a set of 'edit failure' tests, which attempt to write to the wikis -and **should** fail. If there is a bug in pywikibot or MediaWiki, these +and **should** fail. If there is a bug in pywikibot or MediaWiki, these tests **may** actually perform a write operation. These 'edit failure' tests are disabled by default. On Travis they are enabled -- To view, visit https://gerrit.wikimedia.org/r/567290 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1711c8b218f97397937e56d909d1135ff1a0fd24 Gerrit-Change-Number: 567290 Gerrit-PatchSet: 3 Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: [tests] Use wikipedia family for testwiki
by jenkins-bot (Code Review) 30 Jan '20

30 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/568510 ) Change subject: [tests] Use wikipedia family for testwiki ...................................................................... [tests] Use wikipedia family for testwiki Change-Id: If778fba17781f09af5797b8823b36df77ec3f441 --- M tests/data_ingestion_tests.py M tests/deletionbot_tests.py M tests/edit_tests.py M tests/flow_edit_tests.py M tests/flow_tests.py M tests/flow_thanks_tests.py M tests/page_tests.py M tests/pagegenerators_tests.py M tests/protectbot_tests.py M tests/replacebot_tests.py M tests/site_tests.py M tests/thanks_tests.py 12 files changed, 29 insertions(+), 29 deletions(-) Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified diff --git a/tests/data_ingestion_tests.py b/tests/data_ingestion_tests.py index f9e026e..919dbd9 100644 --- a/tests/data_ingestion_tests.py +++ b/tests/data_ingestion_tests.py @@ -118,7 +118,7 @@ """Test TestDataIngestionBot class.""" - family = 'test' + family = 'wikipedia' code = 'test' def test_existing_file(self): diff --git a/tests/deletionbot_tests.py b/tests/deletionbot_tests.py index 1e433c7..3a46f67 100644 --- a/tests/deletionbot_tests.py +++ b/tests/deletionbot_tests.py @@ -19,7 +19,7 @@ """Test deletionbot script.""" - family = 'test' + family = 'wikipedia' code = 'test' sysop = True @@ -52,7 +52,7 @@ """Test deletionbot as a user (not sysop).""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -80,7 +80,7 @@ """Test deletionbot with patching to make it non-write.""" - family = 'test' + family = 'wikipedia' code = 'test' cached = True diff --git a/tests/edit_tests.py b/tests/edit_tests.py index b29ee64..403cbf0 100644 --- a/tests/edit_tests.py +++ b/tests/edit_tests.py @@ -24,7 +24,7 @@ """Run general write tests.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -74,7 +74,7 @@ class TestSiteMergeHistory(TestCase): """Test history merge action.""" - family = 'test' + family = 'wikipedia' code = 'test' write = True diff --git a/tests/flow_edit_tests.py b/tests/flow_edit_tests.py index 4d08b5b..330499a 100644 --- a/tests/flow_edit_tests.py +++ b/tests/flow_edit_tests.py @@ -19,7 +19,7 @@ """Test the creation of Flow topics.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -42,7 +42,7 @@ """Test replying to existing posts.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -159,7 +159,7 @@ """Locking and unlocking topics.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -184,7 +184,7 @@ """Hiding topics and posts.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -224,7 +224,7 @@ """Deleting topics and posts.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -265,7 +265,7 @@ """Suppressing topics and posts.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True @@ -306,7 +306,7 @@ """Flow-related edit failure tests.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True diff --git a/tests/flow_tests.py b/tests/flow_tests.py index ae24720..7a00c5d 100644 --- a/tests/flow_tests.py +++ b/tests/flow_tests.py @@ -158,7 +158,7 @@ """Test errors associated with class methods generating Flow objects.""" - family = 'test' + family = 'wikipedia' code = 'test' cached = True @@ -218,7 +218,7 @@ class TestFlowTopic(TestCase): """Test Topic functions.""" - family = 'test' + family = 'wikipedia' code = 'test' def test_topic(self): diff --git a/tests/flow_thanks_tests.py b/tests/flow_thanks_tests.py index c0b2f4e..7730992 100644 --- a/tests/flow_thanks_tests.py +++ b/tests/flow_thanks_tests.py @@ -20,7 +20,7 @@ """Test thanks for Flow posts.""" - family = 'test' + family = 'wikipedia' code = 'test' write = True diff --git a/tests/page_tests.py b/tests/page_tests.py index fc2d8fc..1b7cbf1 100644 --- a/tests/page_tests.py +++ b/tests/page_tests.py @@ -997,7 +997,7 @@ """Test page delete / undelete actions.""" - family = 'test' + family = 'wikipedia' code = 'test' write = True @@ -1034,7 +1034,7 @@ """Test applicable restriction types.""" - family = 'test' + family = 'wikipedia' code = 'test' def test_applicable_protections(self): @@ -1066,7 +1066,7 @@ """Test page protect / unprotect actions.""" - family = 'test' + family = 'wikipedia' code = 'test' write = True @@ -1157,7 +1157,7 @@ class TestPermalink(DefaultSiteTestCase): """Test that permalink links are correct.""" - family = 'test' + family = 'wikipedia' code = 'test' def test_permalink(self): @@ -1181,7 +1181,7 @@ class TestShortLink(DefaultSiteTestCase): """Test that short link management is correct.""" - family = 'test' + family = 'wikipedia' code = 'test' def test_create_short_link(self): diff --git a/tests/pagegenerators_tests.py b/tests/pagegenerators_tests.py index ba2868d..ace3262 100755 --- a/tests/pagegenerators_tests.py +++ b/tests/pagegenerators_tests.py @@ -335,7 +335,7 @@ """Test SubpageFilterGenerator.""" - family = 'test' + family = 'wikipedia' code = 'test' def test_subpage_filter(self): @@ -362,7 +362,7 @@ """Test PetScanPageGenerator.""" - family = 'test' + family = 'wikipedia' code = 'test' def test_petscan(self): diff --git a/tests/protectbot_tests.py b/tests/protectbot_tests.py index 08b5009..ae00b1e 100644 --- a/tests/protectbot_tests.py +++ b/tests/protectbot_tests.py @@ -19,7 +19,7 @@ """Test ProtectionBot protect/unprotect capabilities.""" - family = 'test' + family = 'wikipedia' code = 'test' sysop = True diff --git a/tests/replacebot_tests.py b/tests/replacebot_tests.py index c533495..0a6f0c4 100644 --- a/tests/replacebot_tests.py +++ b/tests/replacebot_tests.py @@ -31,7 +31,7 @@ 'Press Enter to use this automatic message, or enter a ' 'description of the\nchanges your bot will make:') - family = 'test' + family = 'wikipedia' code = 'test' cached = False diff --git a/tests/site_tests.py b/tests/site_tests.py index 0bfc3a9..279b0f8 100644 --- a/tests/site_tests.py +++ b/tests/site_tests.py @@ -2026,7 +2026,7 @@ """Test site sysop methods that require writing.""" - family = 'test' + family = 'wikipedia' code = 'test' write = True @@ -2170,7 +2170,7 @@ """Test patrol method.""" - family = 'test' + family = 'wikipedia' code = 'test' user = True diff --git a/tests/thanks_tests.py b/tests/thanks_tests.py index 7612cb3..e005c82 100644 --- a/tests/thanks_tests.py +++ b/tests/thanks_tests.py @@ -20,7 +20,7 @@ """Test thanks for revisions.""" - family = 'test' + family = 'wikipedia' code = 'test' write = True @@ -80,7 +80,7 @@ """Test errors when thanking revisions.""" - family = 'test' + family = 'wikipedia' code = 'test' write = -1 -- To view, visit https://gerrit.wikimedia.org/r/568510 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If778fba17781f09af5797b8823b36df77ec3f441 Gerrit-Change-Number: 568510 Gerrit-PatchSet: 3 Gerrit-Owner: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: Remove AS_SYSOP from LoginStatus enum
by jenkins-bot (Code Review) 29 Jan '20

29 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/568058 ) Change subject: Remove AS_SYSOP from LoginStatus enum ...................................................................... Remove AS_SYSOP from LoginStatus enum Bug: T71283 Change-Id: Icde15b081b1620bb4bb4eae4f4324774ed96071d --- M pywikibot/site.py 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified diff --git a/pywikibot/site.py b/pywikibot/site.py index c710057..0ad71fe 100644 --- a/pywikibot/site.py +++ b/pywikibot/site.py @@ -126,7 +126,6 @@ IN_PROGRESS = -2 NOT_LOGGED_IN = -1 AS_USER = 0 - AS_SYSOP = 1 def __repr__(self): """Return internal representation.""" -- To view, visit https://gerrit.wikimedia.org/r/568058 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Icde15b081b1620bb4bb4eae4f4324774ed96071d Gerrit-Change-Number: 568058 Gerrit-PatchSet: 2 Gerrit-Owner: Mpaa <mpaa.wiki(a)gmail.com> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: jenkins-bot (75)

1 0

[Gerrit] pywikibot/core[master]: [doc] Update HISTORY.rst
by jenkins-bot (Code Review) 28 Jan '20

28 Jan '20

jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/567887 ) Change subject: [doc] Update HISTORY.rst ...................................................................... [doc] Update HISTORY.rst Change-Id: Ic0e84ea0de088af0b0cb221dd0216e14966db39c --- M HISTORY.rst 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Dvorapa: Looks good to me, approved jenkins-bot: Verified diff --git a/HISTORY.rst b/HISTORY.rst index fdf865b..8348b67 100644 --- a/HISTORY.rst +++ b/HISTORY.rst @@ -4,6 +4,10 @@ Current release --------------- +* preload urllib.quote() with Python 2 (T243710, T222623) +* Drop isbn_hyphenate package due to outdated data (T243157) +* Fix UnboundLocalError in ProofreadPage._ocr_callback (T243644) +* Deprecate/remove sysop parameter in several methods and functions * Refactor Wikibase entity namespace handling (T160395) * Site.has_api method will be removed in next release * Category.copyTo and Category.copyAndKeep will be removed in the next release -- To view, visit https://gerrit.wikimedia.org/r/567887 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic0e84ea0de088af0b0cb221dd0216e14966db39c Gerrit-Change-Number: 567887 Gerrit-PatchSet: 1 Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: jenkins-bot (75)

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

Pywikibot-commits January 2020