Attachment 138111 Details for Bug 31217 – Bug 31217: OPAC coce.js is not hiding the coce-coverimg div for single-pixel images

[patch] Bug 31217: OPAC coce.js is not hiding the coce-coverimg div for single-pixel images

Bug-31217-OPAC-cocejs-is-not-hiding-the-coce-cover.patch (text/plain), 3.01 KB, created by Alex Buckley on 2022-07-26 05:42:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Buckley

Created: 2022-07-26 05:42:31 UTC

Size: 3.01 KB

patch

obsolete

>From a923a4521c1786cf3686642623fe02014443a5a6 Mon Sep 17 00:00:00 2001
>From: Alex Buckley <alexbuckley@catalyst.net.nz>
>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 %]
>-                                <div class="cover-image" id="coce-coverimg">
>+                                <div class="cover-image coce-coverimg">
>                                     [% IF ( coce_id ) %]
>                                         <a title="Image from Coce" class="[% coce_id | html %]" id="coce-thumbnail-preview"></a>
>                                     [% END %]
>@@ -1506,7 +1506,7 @@
>                                 lightbox_descriptions.push(_("Image from Google Books (<a href='%s'>see the original image</a>)").format($(img).data('link')));
>                             } else if( div.attr("id") == "openlibrary-coverimg" ){
>                                 lightbox_descriptions.push(_("Image from OpenLibrary (<a href='%s'>see the original image</a>)").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

Actions: View | Diff | Splinter Review

Attachments on bug 31217: 137995 | 138070 | 138111 | 138192 | 138198 | 139218 | 139344