Search.pm minor cleanup
authorJoe Atzberger <joe.atzberger@liblime.com>
Wed, 8 Jul 2009 20:27:45 +0000 (15:27 -0500)
committerHenri-Damien LAURENT <henridamien.laurent@biblibre.com>
Wed, 16 Sep 2009 21:19:22 +0000 (23:19 +0200)
commit9ae23c7b05fedc3d5fdc27ef6e2eec8df442c094
tree3c1ffb13b8c7b056daf0e57bbe1c9bbac65e3339
parent965f96e1965da7fe6cbd16290309d38f062a3c7e
Search.pm minor cleanup

Trying to move towards enabling warnings.
Add warn for unknown $query_type.
Use common $dbh.

push @array, ({key1=>value1...}); is the same as
push @array, {key1=>value1...};

if ( scalar(@$arrayref) > 0 ) is the same as
if (scalar @$arrayref)

Lines like:
    my @operators = @$operators if $operators;
are bad because we need @operators to be declared and in scope later,
even if it is undef.  Without $operators, the variable is not in scope.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
Signed-off-by: Henri-Damien LAURENT <henridamien.laurent@biblibre.com>
C4/Search.pm