From daef044b6f876ea2555c4520e6c91319d2760d25 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 image 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 Signed-off-by: Owen Leonard Signed-off-by: Fridolin Somers Signed-off-by: Tomas Cohen Arazi --- 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 0fbbb9442b..06dd33e0e4 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.39.5