Replaced expressions of the form "$x = $x <op> $y" with "$x <op>= $y".
[koha_gimpoz] / C4 / Stats.pm
index df504e3..9828b21 100644 (file)
@@ -1,73 +1,99 @@
-package C4::Stats; #asummes C4/Stats
+package C4::Stats;
 
-#requires DBI.pm to be installed
-#uses DBD:Pg
+# $Id$
+
+# Copyright 2000-2002 Katipo Communications
+#
+# This file is part of Koha.
+#
+# Koha is free software; you can redistribute it and/or modify it under the
+# terms of the GNU General Public License as published by the Free Software
+# Foundation; either version 2 of the License, or (at your option) any later
+# version.
+#
+# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
+# 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
 
 use strict;
 require Exporter;
 use DBI;
-use C4::Database;
-use vars qw($VERSION @ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
+use C4::Context;
+use vars qw($VERSION @ISA @EXPORT);
 
 # set the version for version checking
 $VERSION = 0.01;
 
-@ISA = qw(Exporter);
-@EXPORT = qw(&UpdateStats &statsreport &Count &Overdues &TotalOwing
-&TotalPaid &getcharges &Getpaidbranch &unfilledreserves);
-%EXPORT_TAGS = ( );     # eg: TAG => [ qw!name1 name2! ],
+=head1 NAME
+
+C4::Stats - Update Koha statistics (log)
 
-# your exported package globals go here,
-# as well as any optionally exported functions
+=head1 SYNOPSIS
 
-@EXPORT_OK   = qw($Var1 %Hashit);
+  use C4::Stats;
 
+=head1 DESCRIPTION
 
-# non-exported package globals go here
-use vars qw(@more $stuff);
+The C<&UpdateStats> function adds an entry to the statistics table in
+the Koha database, which acts as an activity log.
 
-# initalize package globals, first exported ones
+=head1 FUNCTIONS
+
+=over 2
+
+=cut
+
+@ISA = qw(Exporter);
+@EXPORT = qw(&UpdateStats &statsreport &Count &Overdues &TotalOwing
+&TotalPaid &getcharges &Getpaidbranch &unfilledreserves);
 
-my $Var1   = '';
-my %Hashit = ();
+=item UpdateStats
 
+  &UpdateStats($env, $branch, $type, $value, $other, $itemnumber,
+               $itemtype, $borrowernumber);
 
-# then the others (which are still accessible as $Some::Module::stuff)
-my $stuff  = '';
-my @more   = ();
+Adds a line to the statistics table of the Koha database. In effect,
+it logs an event.
 
-# all file-scoped lexicals must be created before
-# the functions below that use them.
+C<$branch>, C<$type>, C<$value>, C<$other>, C<$itemnumber>,
+C<$itemtype>, and C<$borrowernumber> correspond to the fields of the
+statistics table in the Koha database.
 
-# file-private lexicals go here
-my $priv_var    = '';
-my %secret_hash = ();
+If C<$branch> is the empty string, the branch code will be taken from
+C<$env-E<gt>{branchcode}>.
 
-# here's a file-private function as a closure,
-# callable as &$priv_func;  it cannot be prototyped.
-my $priv_func = sub {
-  # stuff goes here.
-  };
-  
-# make all your functions, whether exported or not;
+C<$env-E<gt>{usercode}> specifies the value of the C<usercode> field.
 
+=cut
+#'
 sub UpdateStats {
   #module to insert stats data into stats table
   my ($env,$branch,$type,$amount,$other,$itemnum,$itemtype,$borrowernumber)=@_;
-  my $dbh=C4Connect();
-  my $branch=$env->{'branchcode'};
+  my $dbh = C4::Context->dbh;
+  if ($branch eq ''){
+    $branch=$env->{'branchcode'};
+  }
   my $user = $env->{'usercode'};
   print $borrowernumber;
+  # FIXME - Use $dbh->do() instead
   my $sth=$dbh->prepare("Insert into statistics
   (datetime,branch,type,usercode,value,
-  other,itemnumber,itemtype,borrowernumber) 
+  other,itemnumber,itemtype,borrowernumber)
   values (now(),'$branch','$type','$user','$amount',
   '$other','$itemnum','$itemtype','$borrowernumber')");
   $sth->execute;
   $sth->finish;
-  $dbh->disconnect;
 }
 
+# FIXME - Why does this function exist? Why not just rename &circrep
+# to &statsreport?
+# Then again, it only appears to be used in reports.pl which, in turn,
+# doesn't appear to be used. So presumably this function is obsolete.
+# If not, it needs a POD.
 sub statsreport {
   #module to return a list of stats for a given day,time,branch type
   #or to return search stats
@@ -80,13 +106,17 @@ sub statsreport {
   return(@data);
 }
 
+# Only used internally. Probably useless: see comment for
+# &statsreport.
 sub circrep {
   my ($time,$type)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $query="Select * from statistics";
   if ($time eq 'today'){
+    # FIXME - What is this supposed to do? MySQL 3.23.42 barfs on it.
     $query=$query." where type='$type' and datetime
     >=datetime('yesterday'::date)";
+                       # FIXME - .= <<EOT;
   }
   my $sth=$dbh->prepare($query);
   $sth->execute;
@@ -98,14 +128,14 @@ sub circrep {
   }
   $sth->finish;
 #  print $query;
-  $dbh->disconnect;
   return(@results);
-
 }
 
+# FIXME - This is only used in stats.pl, which in turn is never used.
+# Otherwise, this needs a POD.
 sub Count {
   my ($type,$branch,$time,$time2)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $query="Select count(*) from statistics where type='$type'";
   $query.=" and datetime >= '$time' and datetime< '$time2' and branch='$branch'";
   my $sth=$dbh->prepare($query);
@@ -113,50 +143,61 @@ sub Count {
   my $data=$sth->fetchrow_hashref;
   $sth->finish;
 #  print $query;
-  $dbh->disconnect;
   return($data->{'count(*)'});
 }
 
+# FIXME - This function doesn't appear to be used.
+# If it is, it needs a POD.
 sub Overdues{
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $query="Select count(*) from issues where date_due >= now()";
   my $sth=$dbh->prepare($query);
   $sth->execute;
   my $count=$sth->fetchrow_hashref;
   $sth->finish;
-  $dbh->disconnect;
-  return($count->{'count(*)'});  
+  return($count->{'count(*)'});
 }
 
+# FIXME - Never used.
+# Otherwise, it'd need a POD.
 sub TotalOwing{
   my ($type)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $query="Select sum(amountoutstanding) from accountlines";
   if ($type eq 'fine'){
-    $query=$query." where accounttype='F' or accounttype='FN'";
+    $query .= " where accounttype='F' or accounttype='FN'";
   }
   my $sth=$dbh->prepare($query);
 #  print $query;
   $sth->execute;
    my $total=$sth->fetchrow_hashref;
    $sth->finish;
-  $dbh->disconnect; 
   return($total->{'sum(amountoutstanding)'});
 }
 
+# FIXME - Never used.
+# Otherwise, it'd need a POD.
 sub TotalPaid {
   my ($time)=@_;
-  my $dbh=C4Connect;
-  my $query="Select * from statistics,borrowers
-  where statistics.borrowernumber= borrowers.borrowernumber
-  and (statistics.type='payment' or statistics.type='writeoff')";
+  my $dbh = C4::Context->dbh;
+  my $query="Select * from accountlines,borrowers where (accounttype = 'Pay'
+or accounttype ='W')
+  and accountlines.borrowernumber = borrowers.borrowernumber";
   if ($time eq 'today'){
-    $query=$query." and datetime = now()";
+    $query .= " and date = now()";
   } else {
-    $query.=" and datetime > '$time'";
+    $query.=" and date='$time'";
   }
-#  $query.=" order by timestamp";
-  print $query;
+#  my $query="Select * from statistics,borrowers
+#  where statistics.borrowernumber= borrowers.borrowernumber
+#  and (statistics.type='payment' or statistics.type='writeoff') ";
+#  if ($time eq 'today'){
+#    $query=$query." and datetime = now()";
+#  } else {
+#    $query.=" and datetime > '$time'";
+#  }
+  $query.=" order by timestamp";
+#  print $query;
   my $sth=$dbh->prepare($query);
   $sth->execute;
   my @results;
@@ -166,20 +207,21 @@ sub TotalPaid {
     $i++;
   }
    $sth->finish;
-  $dbh->disconnect; 
 #  print $query;
   return(@results);
 }
 
+# FIXME - Only used in stats.pl, which in turn is never used.
+# Otherwise, it needs a POD.
 sub getcharges{
   my($borrowerno,$timestamp)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $timestamp2=$timestamp-1;
   my $query="Select * from accountlines where borrowernumber=$borrowerno
   and timestamp = '$timestamp' and accounttype <> 'Pay' and
   accounttype <> 'W'";
   my $sth=$dbh->prepare($query);
-  print $query,"<br>";
+#  print $query,"<br>";
   $sth->execute;
   my $i=0;
   my @results;
@@ -189,25 +231,30 @@ sub getcharges{
       $i++;
 #    }
   }
-  $dbh->disconnect;
   return(@results);
 }
 
+# This is only used in stats.pl and stats2.pl, neither of which is
+# used.
+# Otherwise, this needs a POD.
 sub Getpaidbranch{
-  my($date)=@_;
-  my $dbh=C4Connect;
-  my $query="select * from statistics where type='payment' and datetime='$date'";
+  my($date,$borrno)=@_;
+  my $dbh = C4::Context->dbh;
+  my $query="select * from statistics where type='payment' and datetime
+  >'$date' and  borrowernumber='$borrno'";
   my $sth=$dbh->prepare($query);
   $sth->execute;
 #  print $query;
   my $data=$sth->fetchrow_hashref;
   $sth->finish;
-  $dbh->disconnect;
   return($data->{'branch'});
 }
 
+# FIXME - This is only used in reservereport.pl and reservereport.xls,
+# neither of which is used.
+# Otherwise, it needs a POD.
 sub unfilledreserves {
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $query="select *,biblio.title from reserves,reserveconstraints,biblio,borrowers,biblioitems where found <> 'F' and cancellationdate
 is NULL and biblio.biblionumber=reserves.biblionumber and
 reserves.constrainttype='o'
@@ -234,14 +281,20 @@ biblio.title,reserves.reservedate";
   $sth=$dbh->prepare($query);
   $sth->execute;
   while (my $data=$sth->fetchrow_hashref){
-    @results[$i]=$data;
+    $results[$i]=$data;
     $i++;
   }
   $sth->finish;
-  $dbh->disconnect;
   return($i,\@results);
 }
 
-END { }       # module clean-up code here (global destructor)
-  
-    
+1;
+__END__
+
+=back
+
+=head1 AUTHOR
+
+Koha Developement team <info@koha.org>
+
+=cut