If someone enters an ISBN13 the link to various site options try to link using this directly rather than first converting it to an ISBN10 (if possible).
All of the navigation links will now get ISBN10, regardless of the pub's original ISBN format. Modified PrintNavBar() in common.py to convert the pub's ISBN to ISBN10 and use the converted result in the link.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
You may want to include a 979 Prefix check in that conversion to ISBN10. ISBN13's with 979 will not convert to ISBN10's (Though we are several (many?) years away from 979's hitting the street).
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
This needs more validation in PrintNavBar - I just got a nasty python error looking at a Magazine with a valid ISSN (specifically, "CLRKSWRLDB2007"). Checking for a leading # isn't enough.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
This is fixed by the check ins of 23-MAY-2009. A pub's ISBN is now converted to an ISBN-10 before being used in the nav bar links.
All of the navigation links will now get ISBN10, regardless of the pub's original ISBN format. Modified PrintNavBar() in common.py to convert the pub's ISBN to ISBN10 and use the converted result in the link.
You may want to include a 979 Prefix check in that conversion to ISBN10. ISBN13's with 979 will not convert to ISBN10's (Though we are several (many?) years away from 979's hitting the street).
This needs more validation in PrintNavBar - I just got a nasty python error looking at a Magazine with a valid ISSN (specifically, "CLRKSWRLDB2007"). Checking for a leading # isn't enough.
Fixed in r2009-02.