Correct POD and misleading variable names from copy/paste laziness
authorJoe Atzberger <joe.atzberger@liblime.com>
Thu, 18 Jun 2009 16:31:32 +0000 (11:31 -0500)
committerGalen Charlton <galen.charlton@liblime.com>
Fri, 19 Jun 2009 11:09:53 +0000 (06:09 -0500)
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
C4/Branch.pm

index 35424d7..a2558d9 100644 (file)
@@ -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