[Koha-bugs] [Bug 12210] Baker & Taylor book jackets not disabled when BakerTaylorBookstoreURL is not populated
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Tue Apr 7 22:02:10 CEST 2015
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12210
Katrin Fischer <katrin.fischer at bsz-bw.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #37530|0 |1
is obsolete| |
--- Comment #11 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
Created attachment 37551
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=37551&action=edit
[PASSED QA] Bug 12210 - Baker & Taylor book jackets not disabled when
BakerTaylorBookstoreURL is not populated
The description of the BakerTaylorBookstoreURL syspref says
"Baker and Taylor "My Library Bookstore" links should be accessed at
https:// isbn (this should be filled in with something like
ocls.mylibrarybookstore.com/MLB/actions/searchHandler.do?nextPage=bookDetails&parentNum=10923&key=).
Leave it blank to disable these links."
Leaving this syspref blank does not disable the links. Instead, they are
populated without the bookstore URL: the cover image for ISBN 1575725738
will cause koha to make a request to http://1575725738, which will, of
course, time out.
Leaving BakerTaylorBookstoreURL blank should disable any URL that
depends on this syspref, as stated in the syspref description.
Test Plan:
1) Apply this patch
2) Enable Baker & Taylor cover images, but *not* BakerTaylorBookstoreURL
3) Perform an OPAC search where you have B&T cover images in the results
4) Note the image links now direct you to the record details
Signed-off-by: Sheila Kearns <sheila.kearns at state.vt.us>
Signed-off-by: Katrin Fischer <katrin.fischer.83 at web.de>
Only affects Baker & Taylor related parts of the code.
Read the code carefully, but couldn't test in the OPAC.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list