From 0a2c7824206f8e6f201b75494b1549ce0a723032 Mon Sep 17 00:00:00 2001 From: Alex Buckley Date: Wed, 10 Aug 2022 15:32:24 +0000 Subject: [PATCH] Bug 31228: Intranet coce.js is not hiding the coce-coverimg div for single-pixel images This is because the intranet detail.tt has coce-coverimg as an ID. Intranet results.tt has coce-coverimg as a class. Intranet coce.js targets coce-coverimg as a class. Intranet detail.tt should be changed to have coce-coverimg as a class. This is inline with the change made to the OPAC in bug 31217 Test plan: 1. Set your system preferences: - CoceHost = https://coce.mykoha.co.nz - CoceProviders = Select all - IntranetCoce = 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 intranet. 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 intranet. Observe no image is displayed. 8. Do an intranet search to get the biblio returned in search results page. Confirm no cover image box is displayed 9. View the intranet search result and detail pages for a different biblio that you know Coce should be displaying the cover image of and confirm that it is correctly displayed with the source of that image listed: For example 'Coce image from Amazon.com' is displayed Sponsored-by: Catalyst IT, New Zealand --- koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt index 0fbbb9442b4..06dd33e0e49 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt @@ -227,7 +227,7 @@ [% IF ( IntranetCoce && CoceProviders && normalized_isbn ) %] [% coce_id = normalized_ean || normalized_isbn %] -
+
[% IF ( coce_id ) %] [% ELSE %] @@ -1340,7 +1340,7 @@ Note that permanent location is a code, and location may be an authval. } else if ( div.attr("id") == "syndetics-bookcoverimg" ){ lightbox_descriptions.push(_("Syndetics cover image (see the original image)").format($(img).data('link'))); } - else if( div.attr("id") == "coce-coverimg" ){ + else if( div.hasClass("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