Bug 7684: (follow-up) fix UTF-8 encoding problems in CSV export
[koha.git] / C4 / Stats.pm
index 8c03061..e1cbd42 100644 (file)
@@ -1,6 +1,5 @@
 package C4::Stats;
 
-# $Id$
 
 # Copyright 2000-2002 Katipo Communications
 #
@@ -15,18 +14,29 @@ package C4::Stats;
 # 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;
 require Exporter;
-use DBI;
 use C4::Context;
+use C4::Debug;
 use vars qw($VERSION @ISA @EXPORT);
 
-# set the version for version checking
-$VERSION = 0.01;
+our $debug;
+
+BEGIN {
+       # set the version for version checking
+    $VERSION = 3.07.00.049;
+       @ISA    = qw(Exporter);
+       @EXPORT = qw(
+               &UpdateStats
+               &TotalPaid
+       );
+}
+
 
 =head1 NAME
 
@@ -45,15 +55,9 @@ the Koha database, which acts as an activity log.
 
 =over 2
 
-=cut
-
-@ISA = qw(Exporter);
-@EXPORT = qw(&UpdateStats &statsreport &TotalOwing
-&TotalPaid &getcharges &Getpaidbranch &unfilledreserves);
-
 =item UpdateStats
 
-  &UpdateStats($env, $branch, $type, $value, $other, $itemnumber,
+  &UpdateStats($branch, $type, $value, $other, $itemnumber,
                $itemtype, $borrowernumber);
 
 Adds a line to the statistics table of the Koha database. In effect,
@@ -63,134 +67,55 @@ 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.
 
-If C<$branch> is the empty string, the branch code will be taken from
-C<$env-E<gt>{branchcode}>.
-
-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 = 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) values (now(),?,?,?,?,?,?,?,?)");
-       $sth->execute($branch,$type,$user,$amount,$other,$itemnum,$itemtype,$borrowernumber);
-       $sth->finish;
+
+    #module to insert stats data into stats table
+    my (
+        $branch,         $type,
+        $amount,   $other,          $itemnum,
+        $itemtype, $borrowernumber, $accountno, $ccode
+      )
+      = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare(
+        "INSERT INTO statistics
+        (datetime, branch, type, value,
+         other, itemnumber, itemtype, borrowernumber, proccode, ccode)
+         VALUES (now(),?,?,?,?,?,?,?,?,?)"
+    );
+    $sth->execute(
+        $branch,    $type,    $amount,
+        $other,     $itemnum, $itemtype, $borrowernumber,
+               $accountno, $ccode
+    );
 }
 
 # Otherwise, it'd need a POD.
 sub TotalPaid {
-       my ($time,$time2)=@_;
-       $time2=$time unless $time2;
-       my $dbh = C4::Context->dbh;
-       my $query="Select * from accountlines,borrowers where (accounttype = 'Pay' or accounttype ='W')
-                                       and accountlines.borrowernumber = borrowers.borrowernumber";
-       my @bind = ();
-       if ($time eq 'today'){
-               $query .= " and date = now()";
-       } else {
-               $query.=" and date>=? and date<=?";
-               @bind = ($time,$time2);
-       }
-       #  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(@bind);
-       my @results;
-       my $i=0;
-       while (my $data=$sth->fetchrow_hashref){
-               $results[$i]=$data;
-               $i++;
-       }
-       $sth->finish;
-       #  print $query;
-       return(@results);
-}
-
-# Otherwise, it needs a POD.
-sub getcharges{
-       my($borrowerno,$timestamp)=@_;
-       my $dbh = C4::Context->dbh;
-       my $timestamp2=$timestamp-1;
-       my $query="";
-       my $sth=$dbh->prepare("Select * from accountlines where borrowernumber=?
-       and timestamp = ? and accounttype <> 'Pay' and
-       accounttype <> 'W'");
-       #  print $query,"<br>";
-       $sth->execute($borrowerno,$timestamp);
-       my $i=0;
-       my @results;
-       while (my $data=$sth->fetchrow_hashref){
-       #    if ($data->{'timestamp'} == $timestamp){
-               $results[$i]=$data;
-               $i++;
-       #    }
-       }
-       return(@results);
-}
-
-# Otherwise, this needs a POD.
-sub Getpaidbranch{
-       my($date,$borrno)=@_;
-       my $dbh = C4::Context->dbh;
-       my $sth=$dbh->prepare("select * from statistics where type='payment' and datetime >? and  borrowernumber=?");
-       $sth->execute($date,$borrno);
-       #  print $query;
-       my $data=$sth->fetchrow_hashref;
-       $sth->finish;
-       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 = C4::Context->dbh;
-       my $sth=$dbh->prepare("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'
-                                                               and (reserves.biblionumber=reserveconstraints.biblionumber
-                                                               and reserves.borrowernumber=reserveconstraints.borrowernumber)
-                                                               and
-                                                               reserves.borrowernumber=borrowers.borrowernumber and
-                                                               biblioitems.biblioitemnumber=reserveconstraints.biblioitemnumber order by
-                                                               biblio.title,reserves.reservedate");
-       $sth->execute;
-       my $i=0;
-       my @results;
-       while (my $data=$sth->fetchrow_hashref){
-               $results[$i]=$data;
-               $i++;
-       }
-       $sth->finish;
-       $sth=$dbh->prepare("select *,biblio.title from reserves,biblio,borrowers where found <> 'F' and cancellationdate
-               is NULL and biblio.biblionumber=reserves.biblionumber and reserves.constrainttype='a' and
-               reserves.borrowernumber=borrowers.borrowernumber
-               order by
-               biblio.title,reserves.reservedate");
-       $sth->execute;
-       while (my $data=$sth->fetchrow_hashref){
-               $results[$i]=$data;
-               $i++;
-       }
-       $sth->finish;
-       return($i,\@results);
+    my ( $time, $time2, $spreadsheet ) = @_;
+    $time2 = $time unless $time2;
+    my $dbh   = C4::Context->dbh;
+    my $query = "SELECT * FROM statistics 
+  LEFT JOIN borrowers ON statistics.borrowernumber= borrowers.borrowernumber
+  WHERE (statistics.type='payment' OR statistics.type='writeoff') ";
+    if ( $time eq 'today' ) {
+        $query .= " AND datetime = now()";
+    } else {
+        $query .= " AND datetime > '$time'";    # FIXME: use placeholders
+    }
+    if ( $time2 ne '' ) {
+        $query .= " AND datetime < '$time2'";   # FIXME: use placeholders
+    }
+    if ($spreadsheet) {
+        $query .= " ORDER BY branch, type";
+    }
+    $debug and warn "TotalPaid query: $query";
+    my $sth = $dbh->prepare($query);
+    $sth->execute();
+    return @{$sth->fetchall_arrayref({})};
 }
 
 1;
@@ -200,6 +125,7 @@ __END__
 
 =head1 AUTHOR
 
-Koha Developement team <info@koha.org>
+Koha Development Team <http://koha-community.org/>
 
 =cut
+