Bug 25336: Show checkouts/fines to guarantor is in the wrong section of the patron...
authorDavid Roberts <david@koha-ptfs.co.uk>
Sun, 3 May 2020 23:48:09 +0000 (23:48 +0000)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 7 Aug 2020 14:54:40 +0000 (16:54 +0200)
This patch moves the "Show checkouts to guarantor" and "Show fines to
guarantor" data to display under the Contact information block

To test:

1) Enable the AllowStaffToSetCheckoutsVisibilityForGuarantor and AllowStaffToSetFinesVisibilityForGuarantor system preferences to Allow
2) View a patron record.
3) The "Show checkouts to guarantor" and "Show fines to guarantor" field
will be on the right hand side of the screen under the Library use
block.
4) Apply patch
5) Check that this data has now moved to the Contact information block
on the left hand side of the patron record.

Works for me!

Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt

index b4351f4..5803963 100644 (file)
                                                 </li>
                                             [% END %]
                                         [% END %]
+                                         <li id="patron-privacyguarantor">
+                                            <span class="label">Show checkouts to guarantor</span>
+                                            [% IF patron.privacy_guarantor_checkouts %]
+                                                Yes
+                                            [% ELSE %]
+                                                No
+                                            [% END %]
+                                        </li>
+
+                                        <li>
+                                            <span class="label">Show fines to guarantor</span>
+                                            [% IF patron.privacy_guarantor_fines %]
+                                                Yes
+                                            [% ELSE %]
+                                                No
+                                            [% END %]
+                                        </li>
+
                                         [% IF guarantees %]
                                             <li>
                                                 <span class="label">Guarantees:</span>
                                             </li>
                                         [% END %]
 
-                                        <li id="patron-privacyguarantor">
-                                            <span class="label">Show checkouts to guarantor</span>
-                                            [% IF patron.privacy_guarantor_checkouts %]
-                                                Yes
-                                            [% ELSE %]
-                                                No
-                                            [% END %]
-                                        </li>
-
-                                        <li>
-                                            <span class="label">Show fines to guarantor</span>
-                                            [% IF patron.privacy_guarantor_fines %]
-                                                Yes
-                                            [% ELSE %]
-                                                No
-                                            [% END %]
-                                        </li>
-
                                         [% IF ( patron.sort1 ) %]
                                             <li id="patron-sort1">
                                                 <span class="label">Sort field 1:</span>