[Koha-patches] [PATCH] Correct POD and misleading variable names from copy/paste laziness

Joe Atzberger joe.atzberger at liblime.com
Thu Jun 18 18:31:32 CEST 2009


---
 C4/Branch.pm |   15 ++++++---------
 1 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/C4/Branch.pm b/C4/Branch.pm
index 35424d7..a2558d9 100644
--- a/C4/Branch.pm
+++ b/C4/Branch.pm
@@ -379,21 +379,18 @@ sub GetBranch ($$) {
 
 =head2 GetBranchDetail
 
-  $branchname = &GetBranchDetail($branchcode);
+    $branch = &GetBranchDetail($branchcode);
 
-Given the branch code, the function returns the corresponding
-branch name for a comprehensive information display
+Given the branch code, the function returns a
+hashref for the corresponding row in the branches table.
 
 =cut
 
 sub GetBranchDetail {
-    my ($branchcode) = @_;
-    my $dbh = C4::Context->dbh;
-    my $sth = $dbh->prepare("SELECT * FROM branches WHERE branchcode = ?");
+    my ($branchcode) = shift or return;
+    my $sth = C4::Context->dbh->prepare("SELECT * FROM branches WHERE branchcode = ?");
     $sth->execute($branchcode);
-    my $branchname = $sth->fetchrow_hashref();
-    $sth->finish();
-    return $branchname;
+    return $sth->fetchrow_hashref();
 }
 
 =head2 get_branchinfos_of
-- 
1.5.6.5




More information about the Koha-patches mailing list