From 640ad36da42868ec64f6fcf342cf32636179505d Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 18 Aug 2015 09:08:09 +0100 Subject: [PATCH] Bug 14685: Fix set guarantor for a child or professional accounts Bug 13970 removed some useful code to automatically set the guarantor when creating a child. But finally there is a better way to do the job, we just need to assume that only children or professional can have a guarantor. Test plan: 1/ Create an adult patron and add a child. 2/ The guarantor info should be retrieved and set on the child creation page. 3/ Create an organization and add a child 4/ The guarantor info should be retrieved and set on the adult creation page. Signed-off-by: Liz Rea This patch catches both cases, thanks Jonathan! Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 438b62e8ffcbab3742ab9a48602a9653b1afde6f) Signed-off-by: Chris Cormack --- members/memberentry.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/members/memberentry.pl b/members/memberentry.pl index 514bdd85da..2b86bff0dd 100755 --- a/members/memberentry.pl +++ b/members/memberentry.pl @@ -238,8 +238,9 @@ if ( ( $op eq 'insert' ) and !$nodouble ) { } #recover all data from guarantor address phone ,fax... -if ( $guarantorid and ( $category_type eq 'C' || $category_type eq 'P' )) { +if ( $guarantorid ) { if (my $guarantordata=GetMember(borrowernumber => $guarantorid)) { + $category_type = $guarantordata->{categorycode} eq 'I' ? 'P' : 'C'; $guarantorinfo=$guarantordata->{'surname'}." , ".$guarantordata->{'firstname'}; $newdata{'contactfirstname'}= $guarantordata->{'firstname'}; $newdata{'contactname'} = $guarantordata->{'surname'}; -- 2.39.5