Bug 33620: Don't show guarantor info on details page if borrowerRelationship is empty

To test:
1. Apply patch
2. Have some values in borrowerRelationship.
3. Go to a patron details page and see both 'Show checkouts to guarantor:' and 'Show charges to guarantor:'
4. Now empty borrowerRelationship.
5. Visit the patron detail page again and you should no longer see either of those fields.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Lucas Gass 2023-07-07 16:07:47 +00:00 committed by Tomas Cohen Arazi
parent f596a47417
commit 3d21b002c1
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F

View file

@ -188,24 +188,25 @@
</li> </li>
[% END %] [% END %]
[% END %] [% END %]
<li id="patron-privacyguarantor"> [% IF Koha.Preference("borrowerRelationship") %]
<span class="label">Show checkouts to guarantor:</span> <li id="patron-privacyguarantor">
[% IF patron.privacy_guarantor_checkouts %] <span class="label">Show checkouts to guarantor:</span>
Yes [% IF patron.privacy_guarantor_checkouts %]
[% ELSE %] Yes
No [% ELSE %]
[% END %] No
</li> [% END %]
</li>
<li id="patron-privacy_guarantor_fines">
<span class="label">Show charges to guarantor:</span>
[% IF patron.privacy_guarantor_fines %]
Yes
[% ELSE %]
No
[% END %]
</li>
<li id="patron-privacy_guarantor_fines">
<span class="label">Show charges to guarantor:</span>
[% IF patron.privacy_guarantor_fines %]
Yes
[% ELSE %]
No
[% END %]
</li>
[% END %]
[% IF guarantees %] [% IF guarantees %]
<li> <li>
<span class="label">Guarantees:</span> <span class="label">Guarantees:</span>