Add some automated tests for 979 ISBNs
Bug #1857060 reported by
Jane Sandberg
This bug affects 17 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Medium
|
Unassigned | ||
3.6 |
Fix Released
|
Medium
|
Unassigned |
Bug Description
In early 2020, libraries will start seeing a bunch more ISBN-13s that start with 979, rather than the classic 978 ISBN-13s. See this announcement: https:/
From a brief spin through our code, it doesn't look like many changes will be needed. Most of the Perl code uses Business:ISBN, which has supported 979 ISBNs since back in 2002.
However, the 979 ISBNs don't have an equivalent ISBN-10. It looks like there are a few places in our business logic where we could be more careful about running as_isbn10, and certainly some more unit tests that we could add.
I'll be able to share a branch soon.
Changed in evergreen: | |
milestone: | none → 3.6-beta |
Changed in evergreen: | |
importance: | Undecided → Medium |
milestone: | 3.6-beta → 3.6-beta2 |
Changed in evergreen: | |
milestone: | 3.6-beta2 → 3.6-rc |
Changed in evergreen: | |
milestone: | 3.6-rc → 3.6.1 |
Changed in evergreen: | |
milestone: | 3.6.1 → 3.6.2 |
summary: |
- A few changes needed to accomodate 979 ISBNs + Add some automated tests for 979 ISBNs |
Changed in evergreen: | |
milestone: | 3.6.2 → 3.6.3 |
Changed in evergreen: | |
milestone: | 3.6.3 → 3.6.4 |
Changed in evergreen: | |
milestone: | 3.6.4 → 3.7.2 |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Here is a branch: user/sandbergja /lp1857060_ tests_for_ 979_isbns
Not adding the pullrequest tag just yet, because I'd like to run it by the new developers group in January.