jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/323676 )
Change subject: isbn.py: convertIsbn10toIsbn13 should not fail for ISBN13
......................................................................
isbn.py: convertIsbn10toIsbn13 should not fail for ISBN13
ISBN13 objects don't have a toISBN13 method, do not try to call that method
on ISBN13 objects.
The patch was originally introduced in a8008a210585759ac46f962eb2d6bd1ebad0460c
by Aadith1996, but reverted to fix minor issues.
Bug: T138911
Change-Id: I47118f6aa18cb659d1f9f96cdab93615e24af07b
---
M scripts/isbn.py
M tests/isbn_tests.py
2 files changed, 10 insertions(+), 1 deletion(-)
Approvals:
jenkins-bot: Verified
Whym: Looks good to me, approved
diff --git a/scripts/isbn.py b/scripts/isbn.py
index f1e1a4b..cfd1fbb 100755
--- a/scripts/isbn.py
+++ b/scripts/isbn.py
@@ -1466,7 +1466,11 @@
except InvalidIsbnException:
# don't change
return isbn
- i13 = getIsbn(isbn).toISBN13()
+ i1x = getIsbn(isbn)
+ if not isinstance(i1x, ISBN13):
+ i13 = i1x.toISBN13()
+ else:
+ i13 = i1x
return i13.code
diff --git a/tests/isbn_tests.py b/tests/isbn_tests.py
index 701c50d..da97687 100644
--- a/tests/isbn_tests.py
+++ b/tests/isbn_tests.py
@@ -138,6 +138,11 @@
'ISBN 978-0-9752298-0-4')
self.assertEqual(convertIsbn10toIsbn13('ISBN 0-9752298-0-1'),
'ISBN 0-9752298-0-1') # Invalid ISBN - no changes
+ # Should not fail for ISBN13
+ self.assertEqual(
+ convertIsbn10toIsbn13('ISBN 978-0-7869-3669-4'),
+ 'ISBN 978-0-7869-3669-4'
+ )
# Errors
isbn = ISBN10('9492098059')
--
To view, visit
https://gerrit.wikimedia.org/r/323676
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I47118f6aa18cb659d1f9f96cdab93615e24af07b
Gerrit-PatchSet: 3
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Dalba <dalba.wiki(a)gmail.com>
Gerrit-Reviewer: Aadith1996 <96aadith(a)gmail.com>
Gerrit-Reviewer: Dalba <dalba.wiki(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Magul <tomasz.magulski(a)gmail.com>
Gerrit-Reviewer: Whym <whym(a)whym.org>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>