From 79625cb15edf49d88d6ed787520ba423541517a1 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 10 Apr 2017 10:23:37 -0300 Subject: [PATCH] Bug 18403: Guarantors Technically a kid from your library group could have a guarantor attached to another group of library, let's deal with this case. Test plan: - Create a kid from your library group - With a superlibrarian staff user create a guarantor that is outside of the group of libraries of the kid - Login with a limited staff user and confirm that on the patron detail page you do not see the link to the guarantor detail page. Note that you see the firstname and surname of the guarantor Q. should it be hidden? --- .../intranet-tmpl/prog/en/modules/members/memberentrygen.tt | 9 ++++++++- members/memberentry.pl | 5 ++++- 2 files changed, 12 insertions(+), 2 deletions(-) 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 ca399b6..9f44423 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt @@ -415,7 +415,14 @@ $(document).ready(function() { [% ELSE %] [% UNLESS nocontactname %]
  • diff --git a/members/memberentry.pl b/members/memberentry.pl index c6d9ccd..755c4f2 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -740,10 +740,13 @@ $template->param( check_member => $check_member,#to know if the borrower already exist(=>1) or not (=>0) "op$op" => 1); +$guarantorid = $borrower_data->{'guarantorid'} || $guarantorid; +my $guarantor = $guarantorid ? Koha::Patrons->find( $guarantorid ) : undef; $template->param( nodouble => $nodouble, borrowernumber => $borrowernumber, #register number - guarantorid => ($borrower_data->{'guarantorid'} || $guarantorid), + guarantor => $guarantor, + guarantorid => $guarantorid, relshiploop => \@relshipdata, btitle=> $default_borrowertitle, guarantorinfo => $guarantorinfo, -- 2.9.3