From 36284b7f5fd6e62f335f159e7776e866957a17bc Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Tue, 6 May 2014 12:26:49 -0400 Subject: [PATCH] Bug 12198: remove unnecessary call to GetMemberIssuesAndFines from guarantor search The script members/guarantor_search.pl made the following call: my ($od,$issue,$fines)=GetMemberIssuesAndFines($results->[$i]{'borrowerid'}); Not only does this call always fail because borrowerid is not a valid key, but the results are never even used in the template! Test Plan: 1) Apply this patch 2) Verify the gurantor search works as before, No changes in behavior should be noted Signed-off-by: Galen Charlton Signed-off-by: Jonathan Druart Signed-off-by: Galen Charlton --- members/guarantor_search.pl | 3 --- 1 file changed, 3 deletions(-) diff --git a/members/guarantor_search.pl b/members/guarantor_search.pl index 59ea5b19de..42d36a7e37 100755 --- a/members/guarantor_search.pl +++ b/members/guarantor_search.pl @@ -71,7 +71,6 @@ if ($member ne ''){ for (my $i=0; $i < $count; $i++){ #find out stats - my ($od,$issue,$fines)=GetMemberIssuesAndFines($results->[$i]{'borrowerid'}); my $guarantorinfo=uc($results->[$i]{'surname'})." , ".ucfirst($results->[$i]{'firstname'}); my %row = ( background => $background, @@ -94,8 +93,6 @@ if ($member ne ''){ dateofbirth =>format_date($results->[$i]{'dateofbirth'}), #fi op - odissue => "$od/$issue", - fines => $fines, borrowernotes => $results->[$i]{'borrowernotes'}); if ( $background ) { $background = 0; } else {$background = 1; } push(@resultsdata, \%row); -- 2.39.5