use strict;
use warnings;
+use JSON qw( to_json );
+
use C4::Context;
use Koha::DateUtils;
+use Koha::Logger;
-use vars qw($VERSION @ISA @EXPORT);
+use vars qw(@ISA @EXPORT);
BEGIN {
- # set the version for version checking
- $VERSION = 3.07.00.049;
require Exporter;
@ISA = qw(Exporter);
@EXPORT = qw(&logaction &cronlogaction &GetLogStatus &displaylog &GetLogs);
#'
sub logaction {
- my ($modulename, $actionname, $objectnumber, $infos)=@_;
+ my ($modulename, $actionname, $objectnumber, $infos, $interface)=@_;
# Get ID of logged in user. if called from a batch job,
# no user session exists and C4::Context->userenv() returns
my $userenv = C4::Context->userenv();
my $usernumber = (ref($userenv) eq 'HASH') ? $userenv->{'number'} : 0;
$usernumber ||= 0;
+ $interface //= C4::Context->interface;
my $dbh = C4::Context->dbh;
- my $sth=$dbh->prepare("Insert into action_logs (timestamp,user,module,action,object,info) values (now(),?,?,?,?,?)");
- $sth->execute($usernumber,$modulename,$actionname,$objectnumber,$infos);
+ my $sth=$dbh->prepare("Insert into action_logs (timestamp,user,module,action,object,info,interface) values (now(),?,?,?,?,?,?)");
+ $sth->execute($usernumber,$modulename,$actionname,$objectnumber,$infos,$interface);
$sth->finish;
+
+ my $logger = Koha::Logger->get(
+ {
+ interface => 'intranet',
+ category => "ActionLogs.$modulename.$actionname"
+ }
+ );
+ $logger->debug(
+ sub {
+ "ACTION LOG: " . to_json(
+ {
+ user => $usernumber,
+ module => $modulename,
+ action => $actionname,
+ object => $objectnumber,
+ info => $infos
+ }
+ );
+ }
+ );
}
=item cronlogaction
my %hash;
$hash{BorrowersLog} = C4::Context->preference("BorrowersLog");
$hash{CataloguingLog} = C4::Context->preference("CataloguingLog");
+ $hash{HoldsLog} = C4::Context->preference("HoldsLog");
$hash{IssueLog} = C4::Context->preference("IssueLog");
$hash{ReturnLog} = C4::Context->preference("ReturnLog");
$hash{SubscriptionLog} = C4::Context->preference("SubscriptionLog");
SELECT action_logs.timestamp, action_logs.action, action_logs.info,
borrowers.cardnumber, borrowers.surname, borrowers.firstname, borrowers.userid,
biblio.biblionumber, biblio.title, biblio.author
- FROM action_logs
- LEFT JOIN borrowers ON borrowers.borrowernumber=action_logs.user
+ FROM action_logs
+ LEFT JOIN borrowers ON borrowers.borrowernumber=action_logs.user
LEFT JOIN biblio ON action_logs.object=biblio.biblionumber
- WHERE action_logs.module = 'cataloguing'
+ WHERE action_logs.module = 'cataloguing'
|;
my %filtermap = ();
if ($modulename eq "catalogue" or $modulename eq "acqui") {
);
} elsif ($modulename eq "members") {
$strsth=qq|
- SELECT action_logs.timestamp, action_logs.action, action_logs.info,
+ SELECT action_logs.timestamp, action_logs.action, action_logs.info,
borrowers.cardnumber, borrowers.surname, borrowers.firstname, borrowers.userid,
bor2.cardnumber, bor2.surname, bor2.firstname, bor2.userid
- FROM action_logs
- LEFT JOIN borrowers ON borrowers.borrowernumber=action_logs.user
+ FROM action_logs
+ LEFT JOIN borrowers ON borrowers.borrowernumber=action_logs.user
LEFT JOIN borrowers as bor2 ON action_logs.object=bor2.borrowernumber
- WHERE action_logs.module = 'members'
+ WHERE action_logs.module = 'members'
|;
%filtermap = (
user => 'borrowers.surname',
$logs = GetLogs($datefrom,$dateto,$user,\@modules,$action,$object,$info);
-Return:
+Return:
C<$logs> is a ref to a hash which containts all columns from action_logs
=cut
my $datefrom = shift;
my $dateto = shift;
my $user = shift;
- my $modules = shift;
+ my $modules = shift;
my $action = shift;
my $object = shift;
my $info = shift;
-
+ my $interfaces = shift;
+
my $iso_datefrom = $datefrom ? output_pref({ dt => dt_from_string( $datefrom ), dateformat => 'iso', dateonly => 1 }) : undef;
my $iso_dateto = $dateto ? output_pref({ dt => dt_from_string( $dateto ), dateformat => 'iso', dateonly => 1 }) : undef;
- my $dbh = C4::Context->dbh;
+ $user ||= q{};
+
+ my $dbh = C4::Context->dbh;
my $query = "
SELECT *
FROM action_logs
";
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 ne "") {
- $query .= " AND user = ? ";
- push(@parameters,$user);
+ $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 ne q{} ) {
+ $query .= " AND user = ? ";
+ push( @parameters, $user );
+ }
+ if ( $modules && scalar(@$modules) ) {
+ $query .=
+ " AND module IN (" . join( ",", map { "?" } @$modules ) . ") ";
+ push( @parameters, @$modules );
}
- if($modules && scalar(@$modules)) {
- $query .= " AND module IN (".join(",",map {"?"} @$modules).") ";
- push(@parameters,@$modules);
+ if ( $action && scalar(@$action) ) {
+ $query .= " AND action IN (" . join( ",", map { "?" } @$action ) . ") ";
+ push( @parameters, @$action );
}
- if($action && scalar(@$action)) {
- $query .= " AND action IN (".join(",",map {"?"} @$action).") ";
- push(@parameters,@$action);
+ if ($object) {
+ $query .= " AND object = ? ";
+ push( @parameters, $object );
}
- if($object) {
- $query .= " AND object = ? ";
- push(@parameters,$object);
+ if ($info) {
+ $query .= " AND info LIKE ? ";
+ push( @parameters, "%" . $info . "%" );
}
- if($info) {
- $query .= " AND info LIKE ? ";
- push(@parameters,"%".$info."%");
+ if ( $interfaces && scalar(@$interfaces) ) {
+ $query .=
+ " AND interface IN (" . join( ",", map { "?" } @$interfaces ) . ") ";
+ push( @parameters, @$interfaces );
}
-
+
my $sth = $dbh->prepare($query);
$sth->execute(@parameters);
-
+
my @logs;
- while( my $row = $sth->fetchrow_hashref ) {
- push @logs , $row;
+ while ( my $row = $sth->fetchrow_hashref ) {
+ push @logs, $row;
}
return \@logs;
}