From 293e00e95c4443d5f03ccc5c2bf7c66228e4f3fb Mon Sep 17 00:00:00 2001
From: Jan Kissig <bibliothek@th-wildau.de>
Date: Wed, 15 Apr 2026 08:28:22 +0000
Subject: [PATCH] Bug 42357: Show patron details on holds table

This patch reintroduces patron details in 'Existing holds' table by using $patron_to_html.

test plan:
a) get a nice fresh ktd and navigate to /cgi-bin/koha/reserve/request.pl?biblionumber=262
b) place a hold for a patron f.e. 23529000035676
c) take a look at the Existing holds tables patron column which only shows the patrons card number.
d) apply patch and run 'yarn js:build'
e) refresh /cgi-bin/koha/reserve/request.pl?biblionumber=262
f) the patron column now should show patrons name
g) set system preference  HidePatronName  to 'Don't show'
h) go to /cgi-bin/koha/reserve/request.pl?biblionumber=262
f) the patron column now should only show the patrons card number.

Signed-off-by: Lawrence O'Regan-Lloyd <loreganlloyd@clamsnet.org>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk>
---
 koha-tmpl/intranet-tmpl/prog/js/holds.js | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/js/holds.js b/koha-tmpl/intranet-tmpl/prog/js/holds.js
index f06cdebd8c..4e94dfef1d 100644
--- a/koha-tmpl/intranet-tmpl/prog/js/holds.js
+++ b/koha-tmpl/intranet-tmpl/prog/js/holds.js
@@ -1535,13 +1535,11 @@ async function load_patron_holds_table(biblio_id, split_data) {
                                 library.name
                             );
                         } else {
-                            return (
-                                '<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=' +
-                                row.patron.patron_id +
-                                '">' +
-                                data +
-                                "</a>"
-                            );
+                            return $patron_to_html(row.patron, {
+                                url: true,
+                                display_cardnumber: true,
+                                hide_patron_name: HidePatronName,
+                            });
                         }
                     },
                 },