Go to file
David Cook 4969a924ee Bug 9737 - Local Covers sometimes appear then disappear on OPAC Search Results/Lists
The current problem:

When the verify_images() function runs from amazonimages.js, it loops
through every img element and selects the elements that come from Amazon
sources.

However, if the height/width is 1 (i.e. Amazon passes off a blank image)
or if the img element's complete property is null or non-existent, then
it searches for the parent node and re-writes the HTML with the "No
Cover Image Available" (NO_AMAZON_IMAGE) text, even though that parent
node might contain other img elements that have images available (from
Google, local images, etc)!

Proposed fix:

Wrap the Amazon image in a span tag (as we do with the Google and local
covers), so that it re-writes the HTML within that parent span, rather
than the parent anchor which contains all the other images.

To test:

1) Turn on the OPACAmazonCoverImages, GoogleJackets, and
   OPACLocalCoverImages system preferences
2) Replace the ISBN in one of your records with 9780615352435
3) Now load a local cover image for that record
4) Search for that record (but make sure you're vague enough to have
   multiple search results)
5) For a split second, you may notice the local cover image appears before being replaced by a "No Cover Image Available".

Now apply this patch

6) Refresh your search results page
7) You should now see your local cover image, PLUS the "No Cover Image
   Available" for the Amazon image, and whatever other graphics/"no
   cover image available" "graphics" for the Google Jackets.

To be comprehensive, try the same set of steps using the OPAC Lists
rather than the OPAC Search Results

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
(cherry picked from commit 2e47475664)

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
2013-04-18 08:25:54 -04:00
C4 Bug 9986: Two fixes for Z3950 search 2013-04-15 08:49:52 -04:00
Koha Bug 9971 - Plugins system is broken! 2013-04-03 06:02:09 -04:00
OpenILS Bug 9239 QA follow-up: remove stray debug code 2013-03-16 21:32:34 -04:00
acqui Bug 9986: Two fixes for Z3950 search 2013-04-15 08:49:52 -04:00
admin Bug 6554: Followup for preferences.pl 2013-04-01 18:51:50 -04:00
authorities Bug 6554 - make Koha internally utf-8 clean 2013-04-01 18:51:49 -04:00
basket Bug 9073 - Download option from the cart should match the menu button in lists 2012-11-25 18:08:46 -05:00
catalogue Bug 6554 - make Koha internally utf-8 clean 2013-04-01 18:51:49 -04:00
cataloguing Bug 9986: Two fixes for Z3950 search 2013-04-15 08:49:52 -04:00
circ Bug 9887 [Revised] Use DataTables on circulation page 2013-04-15 09:10:23 -04:00
debian Bug 10061 - Fix tinymce path references broken by bug 9172 2013-04-18 07:50:46 -04:00
docs Add Aaron Wells to history as committer #207 2013-03-30 20:41:00 -04:00
errors
…
etc Bug 9812 - Forbid access to several files through the browser 2013-04-07 13:43:05 -04:00
install_misc Bug 9914: dependency updates for 3.12 2013-03-23 10:46:07 -04:00
installer Bug 9414 - Fix sample letters for German and Norwegian 2013-04-15 08:54:50 -04:00
koha-tmpl Bug 9737 - Local Covers sometimes appear then disappear on OPAC Search Results/Lists 2013-04-18 08:25:54 -04:00
labels Bug 8943 - Add table sorting to labels - Followup - Sort pulldowns in batch export. 2013-03-31 08:33:42 -04:00
members Bug 6554 - make Koha internally utf-8 clean 2013-04-01 18:51:49 -04:00
misc Bug 8200: Follow up - Removing unused file stats.pl 2013-04-07 13:38:06 -04:00
offline_circ Bug 8220 - Allow koc uploads to go to process queue instead of being applied directly. 2013-03-21 20:35:37 -04:00
opac Bug 7853 - opac-maintanance.pl should correctly redirect back to opac-main.pl, if no updates are outstanding 2013-04-07 13:34:12 -04:00
patroncards Bug 9695 - Use alphabet system preference on page used to browse patrons for patron card batches 2013-03-13 08:43:25 -04:00
plugins Bug 7804 - Add Koha Plugin System - QA Followup 2 2013-03-20 14:50:19 -04:00
reports Bug 6554 - make Koha internally utf-8 clean 2013-04-01 18:51:49 -04:00
reserve Bug 6554 - make Koha internally utf-8 clean 2013-04-01 18:51:49 -04:00
reviews
…
rotating_collections Bug 9605: rotating collections permissions are wrong 2013-03-30 22:11:05 -04:00
selenium
…
serials Bug 6554: Followup for serial search 2013-04-01 18:51:50 -04:00
services Bug 7178: Acquisition item creation improvement 2012-03-26 11:07:23 +02:00
skel Bug 7804 - Add Koha Plugin System 2013-03-20 14:49:47 -04:00
sms
…
suggestion Bug 9226: Wrong branch filter after suggestion creation 2013-02-15 08:34:11 -05:00
svc Bug 9908 - Fixing OCLC Connexion Client 2013-03-29 21:29:10 -04:00
t Bug 9937: Add new unit tests for new routines (introduced by bug 5343) 2013-03-30 17:05:15 -04:00
tags bug 9401: remove direct reads of CGISESSID cookie by JavaScript 2013-02-01 11:05:35 -05:00
test
…
tmp/modified_authorities
…
tools Bug 9905 - Use DataTables on calendar page 2013-04-15 09:13:46 -04:00
virtualshelves Bug 7788: [SIGNED-OFF] Followup: GetShelf call in addbybiblio script corrected 2012-05-22 12:06:26 +02:00
xt Bug 9802 - add test case to ensure man pages have correct XML 2013-03-20 15:11:10 -04:00
.htaccess
…
.mailmap
…
INSTALL Bug 7759, update of install files to use background indexing (and some whitespace tidy) 2012-04-20 16:11:52 +02:00
INSTALL.debian Bug 8092 follow-up: Add optional dependency on CHI 2012-06-09 13:08:18 +02:00
INSTALL.fedora7 Bug 7440 - Remove NoZebra vestiges 2013-03-19 21:17:04 -04:00
INSTALL.opensuse Bug 7759, update of install files to use background indexing (and some whitespace tidy) 2012-04-20 16:11:52 +02:00
INSTALL.ubuntu Bug 8092 follow-up: Add optional dependency on CHI 2012-06-09 13:08:18 +02:00
INSTALL.ubuntu.12.04 Bug 9267 - Ubuntu 12.04 install docs broken 2013-03-07 09:46:55 -05:00
INSTALL.ubuntu.lucid Bug 8092 follow-up: Add optional dependency on CHI 2012-06-09 13:08:18 +02:00
LICENSE Bug 9440 - update Koha's LICENSE file from GPL2 to GPL3 2013-02-12 08:52:10 -05:00
MANIFEST.SKIP Bug 9546 : Updating make manifest tardist 2013-02-06 23:54:46 -05:00
Makefile.PL Bug 9890 - Fix the new plugin system for package installs 2013-03-22 16:54:04 -04:00
README Bug 9440 - update Koha's LICENSE file from GPL2 to GPL3 2013-02-12 08:52:10 -05:00
README.robots
…
about.pl Bug 7440 - Remove NoZebra vestiges 2013-03-19 21:17:04 -04:00
changelanguage.pl
…
edithelp.pl Bug 8622: Fix theme fallback 2012-09-05 11:39:14 +02:00
fix-perl-path.PL
…
help.pl Bug 8949: Trying to view help on a page without any help results in a nasty error 2012-11-05 08:39:23 -05:00
install-CPAN.pl
…
koha_perl_deps.pl Bug 8485 - Make koha_perl_deps.pl batch friendly Added a -b flag for brief which outputs only the perl library name (Foo::BaR), and added a -r flag for required which filters the list to required=Yes perl libraries. 2012-08-31 17:35:49 +02:00
kohaversion.pl Bug 9381: Increment version number 2013-04-07 13:48:46 -04:00
mainpage.pl Revert "Bug 7167: New version for updatedatabase" 2012-12-27 14:02:56 -05:00
rewrite-config.PL Bug 7804 - Add Koha Plugin System 2013-03-20 14:49:47 -04:00

README

Koha is a free software integrated library system.

Koha is distributed under the GNU GPL version 3 or later.
Please read the file LICENSE for more details.

To install or upgrade Koha, please see the INSTALL file appropriate
to your platform.

Report bugs at http://bugs.koha-community.org/

Visit the Koha Project website at http://www.koha-community.org/