Attachment 110783 Details for Bug 26530 – Bug 26530: Use patron card number as checkbox label during patron merge

[patch] Bug 26530: Use patron card number as checkbox label during patron merge

Bug-26530-Use-patron-card-number-as-checkbox-label.patch (text/plain), 2.39 KB, created by Katrin Fischer on 2020-09-26 17:20:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2020-09-26 17:20:27 UTC

Size: 2.39 KB

patch

obsolete

>From 0f1a8a28a15b97abd9d2194c73f127a8530237f7 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>
>---
> koha-tmpl/intranet-tmpl/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.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 26530: 110670 | 110702 | 110783