Summary: | Erroneous display for OPACShelfBrowser with GoogleJackets enabled | ||
---|---|---|---|
Product: | Koha | Reporter: | Indranil Das Gupta <indradg> |
Component: | OPAC | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, claire.hernandez, indradg |
Version: | rel_3_0 | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Screenshot of the error
Proposed fix for 3.0.x |
Description
Indranil Das Gupta
2010-08-15 07:35:13 UTC
Created attachment 2524 [details] [review] Proposed fix for 3.0.x - Copying a couple of changes from HEAD so that the browser uses the browser titles' isbns instead of the page title's isbn. This correction should fix all book cover image providers, not just Google - Correcting system preference check in the template: OPACAmazonEnabled instead of AmazonEnabled (In reply to comment #1) > Created attachment 2524 [details] [review] > Proposed fix for 3.0.x > > - Copying a couple of changes from HEAD so that the browser uses > the browser titles' isbns instead of the page title's isbn. This correction > should fix all book cover image providers, not just Google > - Correcting system preference check in the template: > OPACAmazonEnabled instead of AmazonEnabled the patch is working nicely! This is just for 3.0.x? Henri Damien has this been pushed? I am not yet completly sure but for me, master does not need it. In master code, I found browser_normalized_isbn and OPACAmazonEnabled for example. |