Merge remote-tracking branch 'origin/new/bug_8251'
[koha.git] / reports / acquisitions_stats.pl
index 0b9a19f..7207e64 100755 (executable)
@@ -1,6 +1,5 @@
 #!/usr/bin/perl
 
-# $Id$
 
 # Copyright 2000-2002 Katipo Communications
 #
 # 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.
 
 # test comment
 
 use strict;
+#use warnings; FIXME - Bug 2505
 use C4::Auth;
 use CGI;
 use C4::Context;
-
+use C4::Reports;
 use C4::Output;
 use C4::Koha;
 use C4::Circulation;
+use C4::Dates qw/format_date format_date_in_iso/;
 
 =head1 NAME
 
@@ -36,8 +37,7 @@ plugin that shows a stats on borrowers
 
 =head1 DESCRIPTION
 
-
-=over2
+=over 2
 
 =cut
 
@@ -47,40 +47,36 @@ my $fullreportname = "reports/acquisitions_stats.tmpl";
 my $line           = $input->param("Line");
 my $column         = $input->param("Column");
 my @filters        = $input->param("Filter");
+$filters[0] = format_date_in_iso( $filters[0] );
+$filters[1] = format_date_in_iso( $filters[1] );
+$filters[2] = format_date_in_iso( $filters[2] );
+$filters[3] = format_date_in_iso( $filters[3] );
 my $podsp          = $input->param("PlacedOnDisplay");
 my $rodsp          = $input->param("ReceivedOnDisplay");
 my $aodsp          = $input->param("AcquiredOnDisplay");    ##added by mason.
 my $calc           = $input->param("Cellvalue");
 my $output         = $input->param("output");
 my $basename       = $input->param("basename");
-my $mime           = $input->param("MIME");
-my $del            = $input->param("sep");
 
-#warn "calcul : ".$calc;
 my ($template, $borrowernumber, $cookie)
        = get_template_and_user({template_name => $fullreportname,
                                query => $input,
                                type => "intranet",
                                authnotrequired => 0,
-                               flagsrequired => {reports => 1},
+                               flagsrequired => {reports => '*'},
                                debug => 1,
                                });
+our $sep     = $input->param("sep");
+$sep = "\t" if ($sep eq 'tabulation');
 $template->param(do_it => $do_it,
-               intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"),
-               intranetstylesheet => C4::Context->preference("intranetstylesheet"),
-               IntranetNav => C4::Context->preference("IntranetNav"),
+        DHTMLcalendar_dateformat => C4::Dates->DHTMLcalendar(),
                );
 if ($do_it) {
-
-    #warn
-"line=$line, col=$column, pod=$podsp, rod=$rodsp, aod=$aodsp, calc=$calc, filters=@filters\n";
-
     my $results =
       calculate( $line, $column, $podsp, $rodsp, $aodsp, $calc, \@filters );
     if ( $output eq "screen" ) {
         $template->param( mainloop => $results );
         output_html_with_http_headers $input, $cookie, $template->output;
-        exit(1);
     }
     else {
         print $input->header(
@@ -91,8 +87,6 @@ if ($do_it) {
         );
         my $cols  = @$results[0]->{loopcol};
         my $lines = @$results[0]->{looprow};
-        my $sep;
-        $sep = C4::Context->preference("delimiter");
         print @$results[0]->{line} . "/" . @$results[0]->{column} . $sep;
         foreach my $col (@$cols) {
             print $col->{coltitle} . $sep;
@@ -113,69 +107,57 @@ if ($do_it) {
             print $sep. $col->{totalcol};
         }
         print $sep. @$results[0]->{total};
-        exit(1);
     }
+    exit;
 }
 else {
     my $dbh = C4::Context->dbh;
-    my @values;
-    my %labels;
+    my @select;
     my %select;
     my $req;
-    $req =
-      $dbh->prepare(
-        "SELECT distinctrow id,name FROM aqbooksellers ORDER BY name");
+    $req = $dbh->prepare("SELECT distinctrow id,name FROM aqbooksellers ORDER BY name");
     $req->execute;
     my @select;
     push @select, "";
-
-    #       $select{""}="";
+       $select{''} = "All Suppliers";
     while ( my ( $value, $desc ) = $req->fetchrow ) {
         push @select, $desc;
-
-        #               $select{$value}=$desc;
+        $select{$value}=$desc;
     }
     my $CGIBookSellers = CGI::scrolling_list(
         -name   => 'Filter',
-        -id     => 'Filter',
+        -id     => 'supplier',
         -values => \@select,
-
-        #                               -labels   => \%select,
+        -labels   => \%select,
         -size     => 1,
         -multiple => 0
     );
 
-    $req =
-      $dbh->prepare(
-"SELECT DISTINCTROW itemtype,description FROM itemtypes ORDER BY description"
-      );
+    $req = $dbh->prepare("SELECT DISTINCTROW itemtype,description FROM itemtypes ORDER BY description");
     $req->execute;
     undef @select;
     undef %select;
     push @select, "";
-    $select{""} = "";
+    $select{''} = "All Item Types";
     while ( my ( $value, $desc ) = $req->fetchrow ) {
         push @select, $value;
         $select{$value} = $desc;
     }
     my $CGIItemTypes = CGI::scrolling_list(
         -name     => 'Filter',
-        -id       => 'Filter',
+        -id       => 'itemtypes',
         -values   => \@select,
         -labels   => \%select,
         -size     => 1,
         -multiple => 0
     );
 
-    $req =
-      $dbh->prepare(
-"SELECT DISTINCTROW bookfundid,bookfundname FROM aqbookfund ORDER BY bookfundname"
-      );
+    $req = $dbh->prepare("SELECT DISTINCTROW budget_code, budget_name FROM aqbudgets ORDER BY budget_name");
     $req->execute;
     undef @select;
     undef %select;
     push @select, "";
-    $select{""} = "";
+    $select{''} = "All budgets";
 
     while ( my ( $value, $desc ) = $req->fetchrow ) {
         push @select, $value;
@@ -183,7 +165,7 @@ else {
     }
     my $CGIBudget = CGI::scrolling_list(
         -name     => 'Filter',
-        -id       => 'Filter',
+        -id       => 'budget',
         -values   => \@select,
         -labels   => \%select,
         -size     => 1,
@@ -196,16 +178,22 @@ else {
       );
     $req->execute;
     undef @select;
+    undef %select;
     push @select, "";
+    $select{''} = "All";
     my $hassort1;
     while ( my ($value) = $req->fetchrow ) {
-        $hassort1 = 1 if ($value);
-        push @select, $value;
+               if ($value) {
+                       $hassort1 = 1;
+                       push @select, $value;
+                       $select{$value} = $value;
+               }
     }
     my $CGISort1 = CGI::scrolling_list(
         -name     => 'Filter',
-        -id       => 'Filter',
+        -id       => 'sort1',
         -values   => \@select,
+        -labels   => \%select,
         -size     => 1,
         -multiple => 0
     );
@@ -216,45 +204,38 @@ else {
       );
     $req->execute;
     undef @select;
+    undef %select;
     push @select, "";
+    $select{''} = "All";
     my $hassort2;
     my $hglghtsort2;
 
     while ( my ($value) = $req->fetchrow ) {
-        $hassort2 = 1 if ($value);
-        $hglghtsort2 = !($hassort1);
-        push @select, $value;
+               if ($value) {
+                       $hassort2 = 1;
+                       $hglghtsort2 = !($hassort1);
+                       push @select, $value;
+                       $select{$value} = $value;
+               }
     }
     my $CGISort2 = CGI::scrolling_list(
         -name     => 'Filter',
-        -id       => 'Filter',
+        -id       => 'sort2',
         -values   => \@select,
+        -labels   => \%select,
         -size     => 1,
         -multiple => 0
     );
 
-    my @mime = ( C4::Context->preference("MIME") );
-    foreach my $mime (@mime) {
-
-        #               warn "".$mime;
-    }
-
     my $CGIextChoice = CGI::scrolling_list(
         -name     => 'MIME',
         -id       => 'MIME',
-        -values   => \@mime,
+        -values   => ['CSV'], # FIXME translation
         -size     => 1,
         -multiple => 0
     );
 
-    my @dels         = ( C4::Context->preference("delimiter") );
-    my $CGIsepChoice = CGI::scrolling_list(
-        -name     => 'sep',
-        -id       => 'sep',
-        -values   => \@dels,
-        -size     => 1,
-        -multiple => 0
-    );
+    my $CGIsepChoice = GetDelimiterChoices;
 
     $template->param(
         CGIBookSeller => $CGIBookSellers,
@@ -266,7 +247,8 @@ else {
         CGISort1      => $CGISort1,
         CGISort2      => $CGISort2,
         CGIextChoice  => $CGIextChoice,
-        CGIsepChoice  => $CGIsepChoice
+        CGIsepChoice  => $CGIsepChoice,
+               date_today => C4::Dates->new()->output()
     );
 
 }
@@ -295,77 +277,65 @@ sub calculate {
             if ( ( ( $i == 1 ) or ( $i == 3 ) ) and ( @$filters[ $i - 1 ] ) ) {
                 $cell{err} = 1 if ( @$filters[$i] < @$filters[ $i - 1 ] );
             }
-            $cell{filter} .= @$filters[$i];
+            # format the dates filters, otherwise just fill as is
+            if ($i>=4) {
+                $cell{filter} .= @$filters[$i];
+            } else {
+                $cell{filter} .= format_date(@$filters[$i]);
+            }
             $cell{crit}   .= "Placed On From" if ( $i == 0 );
             $cell{crit}   .= "Placed On To" if ( $i == 1 );
             $cell{crit}   .= "Received On From" if ( $i == 2 );
             $cell{crit}   .= "Received On To" if ( $i == 3 );
 
-            $cell{crit} .= "Acquired On From" if ( $i == 4 );
-            $cell{crit} .= "Acquired On To"   if ( $i == 5 );
+#            $cell{crit} .= "Acquired On From" if ( $i == 4 );
+#            $cell{crit} .= "Acquired On To"   if ( $i == 5 );
 
-            $cell{crit} .= "BookSeller" if ( $i == 6 );
-            $cell{crit} .= "Doc Type"   if ( $i == 7 );
-            $cell{crit} .= "Budget"     if ( $i == 8 );
-            $cell{crit} .= "Sort1"      if ( $i == 9 );
-            $cell{crit} .= "Sort2"      if ( $i == 10 );
+            $cell{crit} .= "BookSeller" if ( $i == 4 );
+            $cell{crit} .= "Doc Type"   if ( $i == 5 );
+            $cell{crit} .= "Budget"     if ( $i == 6 );
+            $cell{crit} .= "Sort1"      if ( $i == 7 );
+            $cell{crit} .= "Sort2"      if ( $i == 8 );
             push @loopfilter, \%cell;
         }
     }
 
     my @linefilter;
 
-    #       warn "filtres ".@filters[0];
-    #       warn "filtres ".@filters[1];
-    #       warn "filtres ".@filters[2];
-    #       warn "filtres ".@filters[3];
-
     $linefilter[0] = @$filters[0] if ( $line =~ /closedate/ );
     $linefilter[1] = @$filters[1] if ( $line =~ /closedate/ );
     $linefilter[0] = @$filters[2] if ( $line =~ /received/ );
     $linefilter[1] = @$filters[3] if ( $line =~ /received/ );
 
-    $linefilter[0] = @$filters[4] if ( $line =~ /acquired/ );
-    $linefilter[1] = @$filters[5] if ( $line =~ /acquired/ );
+    $linefilter[0] = @$filters[4]  if ( $line =~ /bookseller/ );
+    $linefilter[0] = @$filters[5]  if ( $line =~ /itemtype/ );
+    $linefilter[0] = @$filters[6]  if ( $line =~ /budget/ );
+    $linefilter[0] = @$filters[7]  if ( $line =~ /sort1/ );
+    $linefilter[0] = @$filters[8] if ( $line =~ /sort2/ );
 
-    $linefilter[0] = @$filters[6]  if ( $line =~ /bookseller/ );
-    $linefilter[0] = @$filters[7]  if ( $line =~ /itemtype/ );
-    $linefilter[0] = @$filters[8]  if ( $line =~ /bookfund/ );
-    $linefilter[0] = @$filters[9]  if ( $line =~ /sort1/ );
-    $linefilter[0] = @$filters[10] if ( $line =~ /sort2/ );
-
-    #warn "filtre lignes".$linefilter[0]." ".$linefilter[1];
-    #
     my @colfilter;
     $colfilter[0] = @$filters[0] if ( $column =~ /closedate/ );
     $colfilter[1] = @$filters[1] if ( $column =~ /closedate/ );
     $colfilter[0] = @$filters[2] if ( $column =~ /received/ );
     $colfilter[1] = @$filters[3] if ( $column =~ /received/ );
 
-    $colfilter[0] = @$filters[4] if ( $column =~ /acquired/ );
-    $colfilter[1] = @$filters[5] if ( $column =~ /acquired/ );
-
-    $colfilter[0] = @$filters[6]  if ( $column =~ /bookseller/ );
-    $colfilter[0] = @$filters[7]  if ( $column =~ /itemtype/ );
-    $colfilter[0] = @$filters[8]  if ( $column =~ /bookfund/ );
-    $colfilter[0] = @$filters[9]  if ( $column =~ /sort1/ );
-    $colfilter[0] = @$filters[10] if ( $column =~ /sort2/ );
-
-    #warn "filtre col ".$colfilter[0]." ".$colfilter[1];
-
-    #warn "line=$line, podsp=$podsp, rodsp=$rodsp, aodsp=$aodsp\n";
+    $colfilter[0] = @$filters[4]  if ( $column =~ /bookseller/ );
+    $colfilter[0] = @$filters[5]  if ( $column =~ /itemtype/ );
+    $colfilter[0] = @$filters[6]  if ( $column =~ /budget/ );
+    $colfilter[0] = @$filters[7]  if ( $column =~ /sort1/ );
+    $colfilter[0] = @$filters[8]  if ( $column =~ /sort2/ );
 
     # 1st, loop rows.
     my $linefield;
     if ( ( $line =~ /closedate/ ) and ( $podsp == 1 ) ) {
 
         #Display by day
-        $linefield .= "dayname($line)";
+        $linefield .= "concat(hex(weekday($line)+1),'-',dayname($line))";
     }
     elsif ( ( $line =~ /closedate/ ) and ( $podsp == 2 ) ) {
 
         #Display by Month
-        $linefield .= "monthname($line)";
+        $linefield .= "concat(hex(month($line)),'-',monthname($line))";
     }
     elsif ( ( $line =~ /closedate/ ) and ( $podsp == 3 ) ) {
 
@@ -376,34 +346,18 @@ sub calculate {
     elsif ( ( $line =~ /received/ ) and ( $rodsp == 1 ) ) {
 
         #Display by day
-        $linefield .= "dayname($line)";
+        $linefield .= "concat(hex(weekday($line)+1),'-',dayname($line))";
     }
     elsif ( ( $line =~ /received/ ) and ( $rodsp == 2 ) ) {
 
         #Display by Month
-        $linefield .= "monthname($line)";
+        $linefield .= "concat(hex(month($line)),'-',monthname($line))";
     }
     elsif ( ( $line =~ /received/ ) and ( $rodsp == 3 ) ) {
 
         #Display by Year
         $linefield .= "Year($line)";
 
-    }
-    elsif ( ( $line =~ /acquired/ ) and ( $aodsp == 1 ) ) {
-
-        #Display by day
-        $linefield .= "dayname($line)";
-    }
-    elsif ( ( $line =~ /acquired/ ) and ( $aodsp == 2 ) ) {
-
-        #Display by Month
-        $linefield .= "monthname($line)";
-    }
-    elsif ( ( $line =~ /acquired/ ) and ( $aodsp == 3 ) ) {
-
-        #Display by Year
-        $linefield .= "Year($line)";
-
     }
     else {
         $linefield .= $line;
@@ -411,30 +365,30 @@ sub calculate {
 
     my $strsth;
     $strsth .=
-      "SELECT DISTINCTROW $linefield FROM (aqorders, aqbasket,aqorderbreakdown)
-                LEFT JOIN items ON (aqorders.biblioitemnumber= items.biblioitemnumber)
-                LEFT JOIN biblioitems ON (aqorders.biblioitemnumber= biblioitems.biblioitemnumber)
-                LEFT JOIN aqorderdelivery ON (aqorders.ordernumber =aqorderdelivery.ordernumber )
-                LEFT JOIN aqbooksellers ON (aqbasket.booksellerid=aqbooksellers.id) WHERE (aqorders.basketno=aqbasket.basketno)
-                AND (aqorderbreakdown.ordernumber=aqorders.ordernumber) AND $line IS NOT NULL ";
+      "SELECT DISTINCTROW $linefield FROM (aqorders, aqbasket )
+                LEFT JOIN items ON (aqorders.biblionumber= items.biblionumber)
+                LEFT JOIN biblioitems ON (aqorders.biblionumber= biblioitems.biblionumber)
+                LEFT JOIN aqbudgets  ON (aqorders.budget_id = aqbudgets.budget_id )
 
-    if (@linefilter) {
+                LEFT JOIN aqbooksellers ON (aqbasket.booksellerid=aqbooksellers.id) WHERE (aqorders.basketno=aqbasket.basketno)
+                AND $line IS NOT NULL AND $line <> '' ";
+    
+       if (@linefilter) {
         if ( $linefilter[1] ) {
             if ( $linefilter[0] ) {
                 $strsth .= " AND $line BETWEEN ? AND ? ";
             }
             else {
-                $strsth .= " AND $line < ? ";
+                $strsth .= " AND $line <= ? ";
             }
         }
         elsif (
             ( $linefilter[0] )
             and (  ( $line =~ /closedate/ )
-                or ( $line =~ /received/ )
-                or ( $line =~ /acquired/ ) )
+                or ( $line =~ /received/ ))
           )
         {
-            $strsth .= " AND $line > ? ";
+            $strsth .= " AND $line >= ? ";
         }
         elsif ( $linefilter[0] ) {
             $linefilter[0] =~ s/\*/%/g;
@@ -442,13 +396,11 @@ sub calculate {
         }
     }
     $strsth .= " GROUP BY $linefield";
-    $strsth .= " ORDER BY $linefield";
-
-    #warn "377:strsth= $strsth";
+    $strsth .= " ORDER BY $line";
 
     my $sth = $dbh->prepare($strsth);
     if ( (@linefilter) and ( $linefilter[1] ) ) {
-        $sth->execute( "'" . $linefilter[0] . "'", "'" . $linefilter[1] . "'" );
+        $sth->execute( $linefilter[0], $linefilter[1] );
     }
     elsif ( $linefilter[0] ) {
         $sth->execute( $linefilter[0] );
@@ -456,32 +408,25 @@ sub calculate {
     else {
         $sth->execute;
     }
-
-    while ( my ($celvalue) = $sth->fetchrow ) {
-        my %cell;
-        if ($celvalue) {
-            $cell{rowtitle} = $celvalue;
-
-            #               } else {
-            #                       $cell{rowtitle} = "";
-        }
-        $cell{totalrow} = 0;
-        push @loopline, \%cell;
-    }
-
-    #warn "column=$column, podsp=$podsp, rodsp=$rodsp, aodsp=$aodsp\n";
-
+       while ( my ($celvalue) = $sth->fetchrow ) {
+               my %cell;
+               if ($celvalue) {
+                       $cell{rowtitle} = $celvalue;
+                       push @loopline, \%cell;
+               }
+               $cell{totalrow} = 0;
+       }
     # 2nd, loop cols.
     my $colfield;
     if ( ( $column =~ /closedate/ ) and ( $podsp == 1 ) ) {
 
         #Display by day
-        $colfield .= "dayname($column)";
+        $colfield .= "concat(hex(weekday($column)+1),'-',dayname($column))";
     }
     elsif ( ( $column =~ /closedate/ ) and ( $podsp == 2 ) ) {
 
         #Display by Month
-        $colfield .= "monthname($column)";
+        $colfield .= "concat(hex(month($column)),'-',monthname($column))";
     }
     elsif ( ( $column =~ /closedate/ ) and ( $podsp == 3 ) ) {
 
@@ -489,33 +434,17 @@ sub calculate {
         $colfield .= "Year($column)";
 
     }
-    elsif ( ( $column =~ /deliverydate/ ) and ( $rodsp == 1 ) ) {
+    elsif ( ( $column =~ /received/ ) and ( $rodsp == 1 ) ) {
 
         #Display by day
-        $colfield .= "dayname($column)";
+        $colfield .= "concat(hex(weekday($column)+1),'-',dayname($column))";
     }
-    elsif ( ( $column =~ /deliverydate/ ) and ( $rodsp == 2 ) ) {
+    elsif ( ( $column =~ /received/ ) and ( $rodsp == 2 ) ) {
 
         #Display by Month
-        $colfield .= "monthname($column)";
+        $colfield .= "concat(hex(month($column)),'-',monthname($column))";
     }
-    elsif ( ( $column =~ /deliverydate/ ) and ( $rodsp == 3 ) ) {
-
-        #Display by Year
-        $colfield .= "Year($column)";
-
-    }
-    elsif ( ( $column =~ /dateaccessioned/ ) and ( $aodsp == 1 ) ) {
-
-        #Display by day
-        $colfield .= "dayname($column)";
-    }
-    elsif ( ( $column =~ /dateaccessioned/ ) and ( $aodsp == 2 ) ) {
-
-        #Display by Month
-        $colfield .= "monthname($column)";
-    }
-    elsif ( ( $column =~ /dateaccessioned/ ) and ( $aodsp == 3 ) ) {
+    elsif ( ( $column =~ /received/ ) and ( $rodsp == 3 ) ) {
 
         #Display by Year
         $colfield .= "Year($column)";
@@ -527,13 +456,14 @@ sub calculate {
 
     my $strsth2;
     $strsth2 .=
-      "SELECT distinctrow $colfield FROM (aqorders, aqbasket,aqorderbreakdown)
-                 LEFT JOIN items ON (aqorders.biblioitemnumber= items.biblioitemnumber)
-                 LEFT JOIN biblioitems ON (aqorders.biblioitemnumber= biblioitems.biblioitemnumber)
-                 LEFT JOIN aqorderdelivery ON (aqorders.ordernumber =aqorderdelivery.ordernumber )
+      "SELECT distinctrow $colfield FROM (aqorders, aqbasket )
+                 LEFT JOIN items ON (aqorders.biblionumber= items.biblionumber)
+                 LEFT JOIN biblioitems ON (aqorders.biblionumber= biblioitems.biblionumber)
+                 LEFT JOIN aqbudgets  ON (aqorders.budget_id = aqbudgets.budget_id )
+
                  LEFT JOIN aqbooksellers ON (aqbasket.booksellerid=aqbooksellers.id)
-                 WHERE (aqorders.basketno=aqbasket.basketno) AND (aqorderbreakdown.ordernumber=aqorders.ordernumber)
-                 AND $column IS NOT NULL";
+                 WHERE (aqorders.basketno=aqbasket.basketno) AND 
+                 $column IS NOT NULL AND $column <> '' ";
 
     if (@colfilter) {
         if ( $colfilter[1] ) {
@@ -541,33 +471,31 @@ sub calculate {
                 $strsth2 .= " AND $column BETWEEN  ? AND ? ";
             }
             else {
-                $strsth2 .= " AND $column < ? ";
+                $strsth2 .= " AND $column <= ? ";
             }
         }
         elsif (
             ( $colfilter[0] )
             and (  ( $column =~ /closedate/ )
-                or ( $line =~ /received/ )
-                or ( $line =~ /acquired/ ) )
+                or ( $line =~ /received/ ))
           )
         {
-            $strsth2 .= " AND $column > ? ";
+            $strsth2 .= " AND $column >= ? ";
         }
         elsif ( $colfilter[0] ) {
             $colfilter[0] =~ s/\*/%/g;
             $strsth2 .= " AND $column LIKE ? ";
         }
     }
+
+
     $strsth2 .= " GROUP BY $colfield";
     $strsth2 .= " ORDER BY $colfield";
 
-    #        warn "MASON:. $strsth2";
-
     my $sth2 = $dbh->prepare($strsth2);
-    if ( (@colfilter) and ( $colfilter[1] ) ) {
 
-        #                warn "from : ".$colfilter[0]." To  :".$colfilter[1];
-        $sth2->execute( "'" . $colfilter[0] . "'", "'" . $colfilter[1] . "'" );
+    if ( (@colfilter) and ($colfilter[1]) ) {
+        $sth2->execute( $colfilter[0], $colfilter[1] );
     }
     elsif ( $colfilter[0] ) {
         $sth2->execute( $colfilter[0] );
@@ -575,18 +503,13 @@ sub calculate {
     else {
         $sth2->execute;
     }
-
-    while ( my ($celvalue) = $sth2->fetchrow ) {
-        my %cell;
-        if ($celvalue) {
-
-            #               warn "coltitle :".$celvalue;
-            $cell{coltitle} = $celvalue;
-        }
-        push @loopcol, \%cell;
-    }
-
-    #       warn "fin des titres colonnes";
+       while ( my $celvalue = $sth2->fetchrow ) {
+               my %cell;
+               if ($celvalue) {
+                       $cell{coltitle} = $celvalue;
+                       push @loopcol, \%cell;
+               }
+       }
 
     my $i = 0;
     my @totalcol;
@@ -595,11 +518,8 @@ sub calculate {
     #Initialization of cell values.....
     my %table;
 
-    #       warn "init table";
     foreach my $row (@loopline) {
         foreach my $col (@loopcol) {
-
-#                       warn " init table : $row->{rowtitle} / $col->{coltitle} ";
             $table{ $row->{rowtitle} }->{ $col->{coltitle} } = 0;
         }
         $table{ $row->{rowtitle} }->{totalrow} = 0;
@@ -611,60 +531,55 @@ sub calculate {
     $strcalc .= "SUM( aqorders.quantity ) " if ( $process == 1 );
     $strcalc .= "SUM( aqorders.quantity * aqorders.listprice ) "
       if ( $process == 2 );
-    $strcalc .= "FROM (aqorders, aqbasket,aqorderbreakdown)
-                 LEFT JOIN items ON (aqorders.biblioitemnumber= items.biblioitemnumber)
-                 LEFT JOIN biblioitems ON (aqorders.biblioitemnumber= biblioitems.biblioitemnumber)
-                 LEFT JOIN aqorderdelivery ON (aqorders.ordernumber =aqorderdelivery.ordernumber )
-                 LEFT JOIN aqbooksellers ON (aqbasket.booksellerid=aqbooksellers.id) WHERE (aqorders.basketno=aqbasket.basketno)
-                      AND (aqorderbreakdown.ordernumber=aqorders.ordernumber) ";
-
-    @$filters[0] =~ s/\*/%/g if ( @$filters[0] );
-    $strcalc .= " AND aqbasket.closedate > '" . @$filters[0] . "'"
+    $strcalc .= "FROM (aqorders, aqbasket )
+                 LEFT JOIN items ON (aqorders.biblionumber= items.biblionumber)
+                 LEFT JOIN biblioitems ON (aqorders.biblionumber= biblioitems.biblionumber)
+                 LEFT JOIN aqbudgets  ON (aqorders.budget_id = aqbudgets.budget_id )
+
+                 LEFT JOIN aqbooksellers ON (aqbasket.booksellerid=aqbooksellers.id) 
+                 WHERE (aqorders.basketno=aqbasket.basketno) ";
+    
+       @$filters[0] =~ s/\*/%/g if ( @$filters[0] );
+    $strcalc .= " AND aqbasket.closedate >= '" . @$filters[0] . "'"
       if ( @$filters[0] );
     @$filters[1] =~ s/\*/%/g if ( @$filters[1] );
-    $strcalc .= " AND aqbasket.closedate < '" . @$filters[1] . "'"
+    $strcalc .= " AND aqbasket.closedate <= '" . @$filters[1] . "'"
       if ( @$filters[1] );
     @$filters[2] =~ s/\*/%/g if ( @$filters[2] );
-    $strcalc .= " AND aqorderdelivery.deliverydate > '" . @$filters[2] . "'"
+    $strcalc .= " AND aqorders.datereceived >= '" . @$filters[2] . "'"
       if ( @$filters[2] );
     @$filters[3] =~ s/\*/%/g if ( @$filters[3] );
-    $strcalc .= " AND aqorderdelivery.deliverydate < '" . @$filters[3] . "'"
+    $strcalc .= " AND aqorders.datereceived <= '" . @$filters[3] . "'"
       if ( @$filters[3] );
     @$filters[4] =~ s/\*/%/g if ( @$filters[4] );
-    $strcalc .= " AND aqbasket.closedate > '" . @$filters[4] . "'"
+    $strcalc .= " AND aqbooksellers.name LIKE '" . @$filters[4] . "'"
       if ( @$filters[4] );
     @$filters[5] =~ s/\*/%/g if ( @$filters[5] );
-    $strcalc .= " AND aqbasket.closedate < '" . @$filters[5] . "'"
+    $strcalc .= " AND biblioitems.itemtype LIKE '" . @$filters[5] . "'"
       if ( @$filters[5] );
     @$filters[6] =~ s/\*/%/g if ( @$filters[6] );
-    $strcalc .= " AND aqbooksellers.name LIKE '" . @$filters[6] . "'"
+    $strcalc .= " AND aqbudgets.budget_code LIKE '" . @$filters[6] . "'"
       if ( @$filters[6] );
     @$filters[7] =~ s/\*/%/g if ( @$filters[7] );
-    $strcalc .= " AND biblioitems.itemtype LIKE '" . @$filters[7] . "'"
+    $strcalc .= " AND aqorders.sort1 LIKE '" . @$filters[7] . "'"
       if ( @$filters[7] );
     @$filters[8] =~ s/\*/%/g if ( @$filters[8] );
-    $strcalc .= " AND aqbookfund.bookfundid LIKE '" . @$filters[8] . "'"
+    $strcalc .= " AND aqorders.sort2 LIKE '" . @$filters[8] . "'"
       if ( @$filters[8] );
-    @$filters[9] =~ s/\*/%/g if ( @$filters[9] );
-    $strcalc .= " AND aqorders.sort1 LIKE '" . @$filters[9] . "'"
-      if ( @$filters[9] );
-    @$filters[10] =~ s/\*/%/g if ( @$filters[10] );
-    $strcalc .= " AND aqorders.sort2 LIKE '" . @$filters[10] . "'"
-      if ( @$filters[10] );
-    $strcalc .= " GROUP BY $linefield, $colfield ORDER BY $linefield,$colfield";
 
-    #        warn "/n/n". $strcalc;
+    $strcalc .= " AND aqorders.datecancellationprinted is NULL ";
+
+    $strcalc .= " GROUP BY $linefield, $colfield ORDER BY $linefield,$colfield";
     my $dbcalc = $dbh->prepare($strcalc);
     $dbcalc->execute;
 
-    #       warn "filling table";
     my $emptycol;
     while ( my ( $row, $col, $value ) = $dbcalc->fetchrow ) {
+               next if ($row eq undef || $col eq undef);
 
-        #              warn "filling table $row / $col / $value ";
-        $emptycol = 1         if ( $col eq undef );
-        $col      = "zzEMPTY" if ( $col eq undef );
-        $row      = "zzEMPTY" if ( $row eq undef );
+        $emptycol = 1         if ( !defined($col) );
+        $col      = "zzEMPTY" if ( !defined($col) );
+        $row      = "zzEMPTY" if ( !defined($row) );
 
         $table{$row}->{$col}     += $value;
         $table{$row}->{totalrow} += $value;
@@ -675,15 +590,10 @@ sub calculate {
 
     foreach my $row ( sort keys %table ) {
         my @loopcell;
-
         #@loopcol ensures the order for columns is common with column titles
         # and the number matches the number of columns
         foreach my $col (@loopcol) {
-            my $value = $table{$row}->{
-                ( $col->{coltitle} eq "NULL" )
-                ? "zzEMPTY"
-                : $col->{coltitle}
-              };
+            my $value = $table{$row}->{ ( $col->{coltitle} eq "NULL" ) ? "zzEMPTY" : $col->{coltitle} };
             push @loopcell, { value => $value };
         }
         push @looprow,
@@ -696,7 +606,6 @@ sub calculate {
         $hilighted = -$hilighted;
     }
 
-    #       warn "footer processing";
     foreach my $col (@loopcol) {
         my $total = 0;
         foreach my $row (@looprow) {
@@ -707,16 +616,13 @@ sub calculate {
                 ( $col->{coltitle} eq "NULL" ) ? "zzEMPTY"
                 : $col->{coltitle}
               };
-
-#                       warn "value added ".$table{$row->{rowtitle}}->{$col->{coltitle}}. "for line ".$row->{rowtitle};
         }
 
-        #               warn "summ for column ".$col->{coltitle}."  = ".$total;
         push @loopfooter, { 'totalcol' => $total };
     }
 
     # the header of the table
-    #        $globalline{loopfilter}=\@loopfilter;
+       $globalline{loopfilter}=\@loopfilter;
     # the core of the table
     $globalline{looprow} = \@looprow;
     $globalline{loopcol} = \@loopcol;