added autocomplete="off"
[koha.git] / C4 / Log.pm
index 2f791de..6b8ff17 100644 (file)
--- a/C4/Log.pm
+++ b/C4/Log.pm
@@ -4,6 +4,7 @@ package C4::Log;
 
 
 # Copyright 2000-2002 Katipo Communications
+# Copyright 2011 MJ Ray and software.coop
 #
 # This file is part of Koha.
 #
@@ -213,26 +214,21 @@ sub GetLogs {
     my @parameters;
     $query .= " AND DATE_FORMAT(timestamp, '%Y-%m-%d') >= \"".$iso_datefrom."\" " if $iso_datefrom;   #fix me - mysql specific
     $query .= " AND DATE_FORMAT(timestamp, '%Y-%m-%d') <= \"".$iso_dateto."\" " if $iso_dateto;
-    if($user) {
-       $query .= " AND user LIKE ? ";
-       push(@parameters,"%".$user."%");
+    if($user ne "") {
+       $query .= " AND user = ? ";
+       push(@parameters,$user);
     }
-    if(scalar @$modules > 1 or @$modules[0] ne "") {
-           $query .= " AND (1 = 2";  #always false but used to build the query
-           foreach my $module (@$modules) {
-               next if $module eq "";
-               $query .= " or module = ?";
-               push(@parameters,$module);
-           }
-           $query .= ")";
+    if($modules && scalar(@$modules)) {
+       $query .= " AND module IN (".join(",",map {"?"} @$modules).") ";
+       push(@parameters,@$modules);
     }
-    if($action) {
-       $query .= " AND action LIKE ? ";
-       push(@parameters,"%".$action."%");
+    if($action && scalar(@$action)) {
+       $query .= " AND action IN (".join(",",map {"?"} @$action).") ";
+       push(@parameters,@$action);
     }
     if($object) {
-       $query .= " AND object LIKE ? ";
-       push(@parameters,"%".$object."%");
+       $query .= " AND object = ? ";
+       push(@parameters,$object);
     }
     if($info) {
        $query .= " AND info LIKE ? ";
@@ -257,6 +253,6 @@ __END__
 
 =head1 AUTHOR
 
-Koha Development Team <info@koha.org>
+Koha Development Team <http://koha-community.org/>
 
 =cut