From af95e03c083a8ae3d9fcb1749723e798f0ff4a30 Mon Sep 17 00:00:00 2001
From: Pedro Amorim <pedro.amorim@openfifth.co.uk>
Date: Mon, 6 Apr 2026 15:14:00 +0000
Subject: [PATCH] Bug 41983: Display in UI

Test plan:
0) Update the database (ktd, ignore if sandbox):
$ updatedatabase
1) Enable DisplayAddHoldGroups sys prefs
2) Go to 'Henry' and click 'Search to hold':
<staff_url>/cgi-bin/koha/members/moremember.pl?borrowernumber=19
3) Search for 'test' and click the top 'Select all' link
4) Click the downward arrow in the 'Place hold' button and pick 'Place hold for Henry'
5) Set a pickup location for each hold and tick the 'Treat as hold group' checkbox
6) Click 'Place holds'.
7) Run build_holds_queue.pl:
perl misc/cronjobs/holds/build_holds_queue.pl --force
8) Visit the Holds queue page:
<staff_url>/cgi-bin/koha/circ/view_holdsqueue.pl?branchlimit=&itemtypeslimit=&run_report=1
9) Confirm the 'part of a hold group' bit shows and display the modal like 'Holds to pull'

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
---
 .../intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt  | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt
index 0f5524437a..ac92e192e5 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt
@@ -221,6 +221,9 @@
                                                     <td class="hq-title">
                                                         <p> [% INCLUDE 'biblio-title.inc' biblio=itemsloo.biblio link = 1 %] </p>
                                                         <div class="hq-biblionumber content_hidden">[% itemsloo.biblionumber | html %]</div>
+                                                        [% IF ( itemsloo.hold_group_id ) %]
+                                                            <p>(part of a <a class="hold-group" href="/cgi-bin/koha/reserve/hold-group.pl?hold_group_id=[% itemsloo.hold_group_id | html %]">hold group</a>)</p>
+                                                        [% END %]
                                                     </td>
                                                     <td class="hq-author">[% itemsloo.biblio.author | html %]</td>
                                                     <td class="hq-pubdetails">
@@ -272,10 +275,6 @@
                                                             >
                                                             <em>or any available</em>
                                                         [% END %]
-                                                        [% # NOTE: This will never work as hold_group_id doesn't appear in the tmp_holdsqueue table %]
-                                                        [% IF itemsloo.can('hold_group_id') && ( itemsloo.hold_group_id ) %]
-                                                            <div>(part of a <a href="/cgi-bin/koha/reserve/hold-group.pl?hold_group_id=[% itemsloo.hold_group_id | uri %]" class="hold-group">hold group</a>)</div>
-                                                        [% END %]
                                                     </td>
                                                     <td class="hq-patron">
                                                         <p> [% INCLUDE 'patron-title.inc' invert_name=1 patron=itemsloo.patron hide_patron_infos_if_needed=1 link_to="circulation_reserves" %] </p>