From f782476d230cdafd40e187ccb04e6f02eeb4ff19 Mon Sep 17 00:00:00 2001 From: Alex Buckley Date: Mon, 25 Jul 2022 18:49:15 +0000 Subject: [PATCH] Bug 31228: Intraneyt 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 14ac3460a4d..460f0c445db 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.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