From 2e7b86ffba8f3dd407c5d1882b8cc3b90bd8d0dc Mon Sep 17 00:00:00 2001 From: Emmi Takkinen Date: Mon, 25 Apr 2022 10:04:21 +0300 Subject: [PATCH] Bug 29768: Do not hide guarantor name from memberentry.pl Syspref "hidepatronname" currently hides guarantors name on memberentry.pl. This is inconsistent since name isn't hidden e.g. on moremember.pl and staff can already see patrons other personal information during edit. To test: 1) Disable syspref "hidepatronname" 2) Edit guarantee patrons information => Field "Guarantor" only shows guarantors cardnumber (or is empty if guarantor has no cardnumber) 3) Apply patch => Guarantors name is now displayed (as a hyperlink) Also confirm hidepatronname still works as expected on other pages (holds table, items detail page etc.). Sponsored-by: Koha-Suomi Oy --- koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc | 2 +- .../intranet-tmpl/prog/en/modules/members/memberentrygen.tt | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc index e047700963..775c349e6f 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc @@ -51,7 +51,7 @@ [%- SET display_patron_name = 0 -%] [%- END -%] [%- END -%] -[%- IF hide_patron_infos_if_needed AND ( display_patron_name OR display_cardnumber ) -%] +[%- IF display_patron_name OR display_cardnumber -%] [%- IF link_to == 'circulation_reserves' %] [%- ELSIF link_to == 'circulation_recalls' %] [%- ELSE %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt index d01b2316bc..8f84d5b956 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt @@ -440,7 +440,7 @@ legend:hover { [% IF category_type == 'I' %]
  • Organization: - [% INCLUDE 'patron-title.inc' patron=r.guarantor hide_patron_infos_if_needed=1 %] + [% INCLUDE 'patron-title.inc' patron=r.guarantor %]
  • Relationship: @@ -449,7 +449,7 @@ legend:hover { [% ELSE %]
  • Guarantor: - [% INCLUDE 'patron-title.inc' patron=r.guarantor hide_patron_infos_if_needed=1 %] + [% INCLUDE 'patron-title.inc' patron=r.guarantor %]
  • Relationship: -- 2.25.1