From d94f85afe08d1dcfba79605a781585bb9a75d4c6 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Tue, 13 Feb 2024 17:52:33 +0000
Subject: [PATCH] Bug 36044: Show ISBN number of purchase suggestions on the
 OPAC

This patch adds output of ISBN in the table of purchase suggestions in
the OPAC.

The patch also adds some formatting markup to the output of suggestions
information to make it more compact, and corrects the label on the
suggestion titles so that they work to trigger the corresponding
checkbox.

To test, apply the patch and submit some suggestions. Include ISBN with
some requests.

Log in to the OPAC and view your suggestions. Confirm that the
information is displayed well, including ISBN.
---
 .../bootstrap/en/modules/opac-suggestions.tt  | 59 ++++++++++++++-----
 1 file changed, 44 insertions(+), 15 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
index 2997f20583..828d57e741 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-suggestions.tt
@@ -426,22 +426,51 @@
                                                         </td>
                                                     [% END %]
                                                     <td>
-                                                        <p>
-                                                            <label for="id[% suggestions_loo.suggestionid | html %]">
-                                                                [% IF suggestion.biblionumber %]
-                                                                    <strong><a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% suggestion.biblionumber | uri %]">[% suggestion.title | html %]</a></strong>
-                                                                [% ELSE %]
-                                                                    <strong>[% suggestion.title | html %]</strong>
+                                                        <label for="id[% suggestion.suggestionid | html %]">
+                                                            [% IF suggestion.biblionumber %]
+                                                                <strong><a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% suggestion.biblionumber | uri %]">[% suggestion.title | html %]</a></strong>
+                                                            [% ELSE %]
+                                                                <strong>[% suggestion.title | html %]</strong>
+                                                            [% END %]
+                                                        </label>
+                                                        [% IF ( suggestion.author ) %]
+                                                            <span class="results_summary">
+                                                                <span class="label">Author:</span>
+                                                                [% suggestion.author | html %]
+                                                            </span>
+                                                        [% END %]
+                                                        [% IF ( suggestion.publishercode || suggestion.place || suggestion.copyrightdate ) %]
+                                                            <span class="results_summary">
+                                                                <span class="label">Publication details:</span>
+                                                                [% IF ( suggestion.publishercode ) %]
+                                                                    [% suggestion.publishercode | html %]
+                                                                [% END %]
+                                                                [% IF ( suggestion.place ) %]
+                                                                    ([% suggestion.place | html %])
+                                                                [% END %]
+                                                                [% IF ( suggestion.copyrightdate ) %]
+                                                                    [% suggestion.copyrightdate | html %]
                                                                 [% END %]
-                                                            </label>
-                                                        </p>
-                                                            <p>[% IF ( suggestion.author ) %][% suggestion.author | html %],[% END %]
-                                                                [% IF ( suggestion.copyrightdate ) %] - [% suggestion.copyrightdate | html %],[% END %]
-                                                                [% IF ( suggestion.publishercode ) %] - [% suggestion.publishercode | html %][% END %]
-                                                                [% IF ( suggestion.place ) %]([% suggestion.place | html %])[% END %]
-                                                                [% IF ( suggestion.collectiontitle ) %] , [% suggestion.collectiontitle | html %][% END %]
-                                                                [% IF ( suggestion.itemtype ) %] - [% AuthorisedValues.GetByCode( 'SUGGEST_FORMAT', suggestion.itemtype, 1 ) | html %][% END %]
-                                                        </p>
+                                                            </span>
+                                                        [% END %]
+                                                        [% IF ( suggestion.collectiontitle ) %]
+                                                            <span class="results_summary">
+                                                                <span class="label">Collection:</span>
+                                                                [% suggestion.collectiontitle | html %]
+                                                            </span>
+                                                        [% END %]
+                                                        [% IF ( suggestion.itemtype ) %]
+                                                            <span class="results_summary">
+                                                                <span class="label">Item type:</span>
+                                                                [% AuthorisedValues.GetByCode( 'SUGGEST_FORMAT', suggestion.itemtype, 1 ) | html %]
+                                                            </span>
+                                                        [% END %]
+                                                        [% IF ( suggestion.isbn ) %]
+                                                            <span class="results_summary">
+                                                                <span class="label">ISBN:</span>
+                                                                [% suggestion.isbn | html %]
+                                                            </span>
+                                                        [% END %]
                                                     </td>
                                                     <td data-order="[% suggestion.suggesteddate | html %]">
                                                         [% IF ( suggestion.suggesteddate ) %][% suggestion.suggesteddate |$KohaDates %][% END %]
-- 
2.30.2