# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
# A PARTICULAR PURPOSE. See the GNU General Public License for more details.
#
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA 02111-1307 USA
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
use strict;
use warnings;
$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."%");
+ $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 .= ")";
+ $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 ? ";
=head1 AUTHOR
-Koha Developement team <info@koha.org>
+Koha Development Team <http://koha-community.org/>
=cut