LP1791791: Google book previews not displaying from a bib linked from a search.
authorGarry Collum <gcollum@gmail.com>
Sun, 19 Dec 2021 01:31:22 +0000 (20:31 -0500)
committerGalen Charlton <gmc@equinoxOLI.org>
Wed, 26 Apr 2023 19:57:00 +0000 (19:57 +0000)
The javascript that queries Google Books Preview looks for ISBN(s) wrapped in
a rdetail_value class.  There was a line of code in which the isbn variable
 was not enclosed in this class.  This patch just wraps that stray variable
into a <span> with the class.

To test:
0. Use the TPAC skin - this bug does not affect the Bootstrap skin
1. Perform a keyword search for an bib that should have a google book's link.
   As of March 2023, ISBN 9780786496570 has previews enabled.
2. Go to the bib display and notice that there is no link.
3. Remove the ';query=something' text from the url and notice that the
google book information displays.
4. Apply the patch.
5. Perform the same search.  The google book information should now display.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Open-ILS/src/templates/opac/parts/record/summary.tt2

index b6e4f25..452f987 100644 (file)
@@ -335,7 +335,8 @@ END;
 <ul>
     [%- IF attrs.hl.isbn.size; FOR isbn IN attrs.hl.isbn %]
     <li class='rdetail_isbns'>
-        <strong class='rdetail_label'>[% l('ISBN:'); %]</strong> [% isbn %]
+        <strong class='rdetail_label'>[% l('ISBN:'); %]</strong> 
+        <span class='rdetail_value' property='isbn'>[% isbn | html  %]</span>
     </li>
     [%- END; ELSIF attrs.isbns.0;
           FOR isbn IN attrs.isbns;