From a923a4521c1786cf3686642623fe02014443a5a6 Mon Sep 17 00:00:00 2001 From: Alex Buckley Date: Tue, 26 Jul 2022 16:38:29 +1200 Subject: [PATCH] Bug 31217: OPAC coce.js is not hiding the coce-coverimg div for single-pixel images This is because opac-detail.tt has coce-coverimg as an ID, but OPAC coce.js targets coce-coverimg as a class. opac-detail.tt should be changed to have coce-coverimg as a class. Test plan: 1. Set your system preferences: - CoceHost = https://coce.mykoha.co.nz - CoceProviders = Select all - OpacCoce = Enable 2. Add the ISBN of '9780262336499' to a biblio 3. Open a "Incognito" window in your browser (recommended to use Google Chrome to test this) 4. View the biblio detail page in the OPAC. Observe a 'Coce image from Amazon' box loads with a spinner 5. Apply patch and restart services 6. Clear your browser cache 7. Reload the same biblio detail page in the OPAC. Observe no image is displayed. 8. View another OPAC biblio detail page that does have a successfully loading coce image and confirm the source of that image is listed: For example 'Coce image from Amazon.com' is displayed Sponsored-by: Toi Ohomai Institute of Technology, New Zealand --- koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt index 7c30dafda96..82fbc661b11 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt @@ -102,7 +102,7 @@ [% IF ( Koha.Preference('OPACCoce') && Koha.Preference('CoceProviders') && normalized_isbn ) %] [% coce_id = normalized_ean || normalized_isbn %] -
+
[% IF ( coce_id ) %] [% END %] @@ -1506,7 +1506,7 @@ lightbox_descriptions.push(_("Image from Google Books (see the original image)").format($(img).data('link'))); } else if( div.attr("id") == "openlibrary-coverimg" ){ lightbox_descriptions.push(_("Image from OpenLibrary (see the original image)").format($(img).data('link'))); - } else if( div.attr("id") == "coce-coverimg" ){ + } else if( div.attr("class") == "cover-image coce-coverimg" ){ // Identify which service's image is being loaded by Coce var coce_description; if( $(img).attr("src").indexOf('amazon.com') >= 0 ){ -- 2.20.1