Bug 7061: unnecessary global variables declared in C4::SQLHelper
authorIan Walls <ian.walls@bywatersolutions.com>
Wed, 19 Oct 2011 00:17:17 +0000 (20:17 -0400)
committerPaul Poulain <paul.poulain@biblibre.com>
Fri, 4 Nov 2011 11:11:13 +0000 (12:11 +0100)
Removes $tablename and $hash from global scope of C4/SQLHelper.pm.  They are never used;
all instances of $table are locally scoped to the subroutines, and $hash is never used.

This may help with persistance; if not, it's at least a bit of trivial cleanup.

To test:

1.  Search for a patron
2.  Create a Contract in Acquisitions
3.  Modify that Contract
4.  Delete that Contract

All instances should return error free.  This checks the subroutines of SQLHelper

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
C4/SQLHelper.pm

index 285f7d2..4c74df2 100644 (file)
@@ -43,9 +43,6 @@ BEGIN {
                                );
 }
 
-my $tablename;
-my $hash;
-
 =head1 NAME
 
 C4::SQLHelper - Perl Module containing convenience functions for SQL Handling