[Koha-patches] [PATCH] [SIGNED-OFF] Bug 3319 Followup: Fix compatibility with fix for bug 4945

Nicole C. Engard nengard at bywatersolutions.com
Sat Feb 19 11:38:32 CET 2011


From: Ian Walls <ian.walls at bywatersolutions.com>

The fix for 3319 overwrote the @branchloop variable with output from GetBranchesLoop,
which forces a selected branch.  Removing the extra call, and just measuring the size of
@branchloop as it was build, plus some dereferencing, fixes the issue.

Signed-off-by: Nicole C. Engard <nengard at bywatersolutions.com>
---
 members/members-home.pl |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/members/members-home.pl b/members/members-home.pl
index 6cb60fa..00c6034 100755
--- a/members/members-home.pl
+++ b/members/members-home.pl
@@ -54,13 +54,12 @@ foreach (sort { $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname}
 my @categories;
 my $no_categories;
 my $no_add = 0;
-my $branchloop = (defined $branch?GetBranchesLoop($branch):GetBranchesLoop());
-if(scalar(@$branchloop) < 1){
+if(scalar(@branchloop) < 1){
     $no_add = 1;
     $template->param(no_branches => 1);
 } 
 else {
-    $template->param(branchloop=>\@$branchloop);
+    $template->param(branchloop=>\@branchloop);
 }
 
 @categories=C4::Category->all;
-- 
1.7.2.3



More information about the Koha-patches mailing list