From 41f0f22cb2f13d04ad9422b73bddc453591e4732 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 11 Sep 2020 16:32:29 +0000
Subject: [PATCH] Bug 26530: Use patron card number as checkbox label during
 patron merge

This patch makes a minor change the the merge patrons template so that
on the patron selection stage the user can click the card number to
select the radio button.

To test, apply the patch and perform a patron search in the patrons
module.

- In patron search results, select two patrons to merge.
- Click the "Merge selected patrons" button.
- On the page showing the two patrons to merge, click each of the card
  numbers to confirm that it selects the correct radio button.

Signed-off-by: Jennifer <jennifer.marlatt@coronaca.gov>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 .../prog/en/modules/members/merge-patrons.tt              | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/merge-patrons.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/merge-patrons.tt
index 94abaa34e6..4d9680e479 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/merge-patrons.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/merge-patrons.tt
@@ -70,8 +70,12 @@
                             <tbody>
                                 [% FOREACH p IN patrons %]
                                     <tr>
-                                        <td><input class='keeper' type='radio' name='keeper' value='[% p.id | html %]' /></td>
-                                        <td>[% p.cardnumber | html %]</td>
+                                        <td><input class="keeper" type="radio" name="keeper" value="[% p.id | html %]" id="keeper_[% p.id | html %]" /></td>
+                                        <td>
+                                            <label for="keeper_[% p.id | html %]">
+                                                <strong>[% p.cardnumber | html %]</strong>
+                                            </label>
+                                        </td>
                                         <td>[% INCLUDE 'patron-title.inc' patron = p %]</td>
                                         <td>[% p.dateofbirth | $KohaDates %]</td>
                                         <td>[% Categories.GetName( p.categorycode ) | html %] ([% p.categorycode | html %])</td>
-- 
2.39.5