X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=reports%2Fcat_issues_top.pl;h=604258e9c98530b7832b1b6f98f832adb07e6018;hb=refs%2Fheads%2Fkoha_ffzg;hp=d033b492c3c79d05b065fc1b3b5c63cdd09280df;hpb=a5bd2403218b5add150088ab525b1f00e50a0c12;p=koha.git diff --git a/reports/cat_issues_top.pl b/reports/cat_issues_top.pl index d033b492c3..604258e9c9 100755 --- a/reports/cat_issues_top.pl +++ b/reports/cat_issues_top.pl @@ -1,34 +1,34 @@ #!/usr/bin/perl -# $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 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 3 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. +# 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 +# You should have received a copy of the GNU General Public License +# along with Koha; if not, see . -use strict; +use Modern::Perl; use C4::Auth; -use CGI; +use CGI qw ( -utf8 ); use C4::Context; -use C4::Branch; # GetBranches use C4::Output; use C4::Koha; use C4::Circulation; -use C4::Date; +use C4::Reports; use C4::Members; +use Koha::DateUtils; +use Koha::ItemTypes; =head1 NAME @@ -36,33 +36,32 @@ plugin that shows a stats on borrowers =head1 DESCRIPTION -=over 2 - =cut my $input = new CGI; my $do_it=$input->param('do_it'); -my $fullreportname = "reports/cat_issues_top.tmpl"; +my $fullreportname = "reports/cat_issues_top.tt"; my $limit = $input->param("Limit"); my $column = $input->param("Criteria"); -my @filters = $input->param("Filter"); -$filters[0]=format_date_in_iso($filters[0]); -$filters[1]=format_date_in_iso($filters[1]); +my @filters = $input->multi_param("Filter"); +foreach ( @filters[0..3] ) { + $_ and $_ = eval { output_pref( { dt => dt_from_string ( $_ ), dateonly => 1, dateformat => 'iso' } ); }; +} + 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, - DHTMLcalendar_dateformat => get_date_format_string_for_DHTMLcalendar(), ); if ($do_it) { # Displaying results @@ -72,7 +71,7 @@ if ($do_it) { $template->param(mainloop => $results, limit => $limit); output_html_with_http_headers $input, $cookie, $template->output; - exit(1); + exit; } else { # Printing to a csv file print $input->header(-type => 'application/vnd.sun.xml.calc', @@ -81,8 +80,6 @@ if ($do_it) { -filename=>"$basename.csv" ); my $cols = @$results[0]->{loopcol}; my $lines = @$results[0]->{looprow}; - my $sep; - $sep =C4::Context->preference("delimiter"); # header top-right print @$results[0]->{line} ."/". @$results[0]->{column} .$sep; # Other header @@ -96,90 +93,55 @@ if ($do_it) { print $line->{rowtitle}.$sep; foreach my $cell (@$x) { print $cell->{value}.$sep; + print $cell->{count} // ''; } - print $line->{totalrow}; print "\n"; } -# footer - print "TOTAL"; - $cols = @$results[0]->{loopfooter}; - foreach my $col ( @$cols ) { - print $sep.$col->{totalcol}; - } - print $sep.@$results[0]->{total}; - exit(1); + exit; } # Displaying choices } else { my $dbh = C4::Context->dbh; - my @values; - my %labels; - my %select; - my $req; - my @mime = ( C4::Context->preference("MIME") ); -# foreach my $mime (@mime){ -# warn "".$mime; -# } - - my $CGIextChoice=CGI::scrolling_list( - -name => 'MIME', - -id => 'MIME', - -values => \@mime, - -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 ); - #branch - my $branches = GetBranches; - my @branchloop; - foreach my $thisbranch (keys %$branches) { -# my $selected = 1 if $thisbranch eq $branch; - my %row =(value => $thisbranch, -# selected => $selected, - branchname => $branches->{$thisbranch}->{'branchname'}, - ); - push @branchloop, \%row; - } + my $CGIextChoice = ( 'CSV' ); # FIXME translation + my $CGIsepChoice=GetDelimiterChoices; #doctype - my $itemtypes = GetItemTypes; - my @itemtypeloop; - foreach my $thisitemtype (keys %$itemtypes) { -# my $selected = 1 if $thisbranch eq $branch; - my %row =(value => $thisitemtype, -# selected => $selected, - description => $itemtypes->{$thisitemtype}->{'description'}, + my $itemtypes = Koha::ItemTypes->search_with_localization; + + #ccode + my $ccodes = GetAuthorisedValues('CCODE'); + my @ccodeloop; + for my $thisccode (@$ccodes) { + my %row = (value => $thisccode->{authorised_value}, + description => $thisccode->{lib}, ); - push @itemtypeloop, \%row; + push @ccodeloop, \%row; } - - #borcat - my ($codes,$labels) = GetborCatFromCatType(undef,undef); - my @borcatloop; - foreach my $thisborcat (sort keys %$labels) { -# my $selected = 1 if $thisbranch eq $branch; - my %row =(value => $thisborcat, -# selected => $selected, - description => $labels->{$thisborcat}, + + @ccodeloop = sort {$a->{value} cmp $b->{value}} @ccodeloop; + + #shelvingloc + my $shelvinglocs = GetAuthorisedValues('LOC'); + my @shelvinglocloop; + for my $thisloc (@$shelvinglocs) { + my %row = (value => $thisloc->{authorised_value}, + description => $thisloc->{lib}, ); - push @borcatloop, \%row; + push @shelvinglocloop, \%row; } - - #Day - #Month + + @shelvinglocloop = sort {$a->{value} cmp $b->{value}} @shelvinglocloop; + + my $patron_categories = Koha::Patron::Categories->search_limited({}, {order_by => ['categorycode']}); + $template->param( CGIextChoice => $CGIextChoice, CGIsepChoice => $CGIsepChoice, - branchloop =>\@branchloop, - itemtypeloop =>\@itemtypeloop, - borcatloop =>\@borcatloop, + itemtypes => $itemtypes, + ccodeloop =>\@ccodeloop, + shelvinglocloop =>\@shelvinglocloop, + patron_categories => $patron_categories, ); output_html_with_http_headers $input, $cookie, $template->output; } @@ -190,9 +152,7 @@ output_html_with_http_headers $input, $cookie, $template->output; sub calculate { my ($line, $column, $filters) = @_; my @mainloop; - my @loopfooter; my @loopcol; - my @loopline; my @looprow; my %globalline; my $grantotal =0; @@ -203,7 +163,7 @@ sub calculate { # Checking filters # my @loopfilter; - for (my $i=0;$i<=6;$i++) { + for (my $i=0;$i<=12;$i++) { my %cell; if ( @$filters[$i] ) { if (($i==1) and (@$filters[$i-1])) { @@ -213,25 +173,32 @@ sub calculate { if ($i>=2) { $cell{filter} .= @$filters[$i]; } else { - $cell{filter} .= format_date(@$filters[$i]); - } $cell{crit} .="Issue From" if ($i==0); + $cell{filter} .= eval { output_pref( { dt => dt_from_string( @$filters[$i] ), dateonly => 1 }); } + if ( @$filters[$i] ); + } + $cell{crit} .="Issue From" if ($i==0); $cell{crit} .="Issue To" if ($i==1); $cell{crit} .="Return From" if ($i==2); $cell{crit} .="Return To" if ($i==3); $cell{crit} .="Branch" if ($i==4); $cell{crit} .="Doc Type" if ($i==5); - $cell{crit} .="Bor Cat" if ($i==6); - $cell{crit} .="Day" if ($i==7); - $cell{crit} .="Month" if ($i==8); - $cell{crit} .="Year" if ($i==9); + $cell{crit} .="Call number" if ($i==6); + $cell{crit} .="Collection code" if ($i==7); + $cell{crit} .="Shelving location" if ($i==8); + $cell{crit} .="Bor Cat" if ($i==9); + $cell{crit} .="Day" if ($i==10); + $cell{crit} .="Month" if ($i==11); + $cell{crit} .="Year" if ($i==12); push @loopfilter, \%cell; } } my $colfield; my $colorder; if ($column){ - $column = "issues.".$column if (($column=~/branchcode/) or ($column=~/timestamp/)); - $column = "biblioitems.".$column if $column=~/itemtype/; + $column = "old_issues.".$column if (($column=~/branchcode/) or ($column=~/timestamp/)); + if($column=~/itemtype/){ + $column = C4::Context->preference('item-level_itypes') ? "items.itype": "biblioitems.itemtype"; + } $column = "borrowers.".$column if $column=~/categorycode/; my @colfilter ; $colfilter[0] = @$filters[0] if ($column =~ /timestamp/ ) ; @@ -240,27 +207,31 @@ sub calculate { $colfilter[1] = @$filters[3] if ($column =~ /returndate/ ) ; $colfilter[0] = @$filters[4] if ($column =~ /branch/ ) ; $colfilter[0] = @$filters[5] if ($column =~ /itemtype/ ) ; - $colfilter[0] = @$filters[6] if ($column =~ /category/ ) ; - # $colfilter[0] = @$filters[11] if ($column =~ /sort2/ ) ; - $colfilter[0] = @$filters[7] if ($column =~ /timestamp/ ) ; - $colfilter[0] = @$filters[8] if ($column =~ /timestamp/ ) ; + # These limits does not currently exist, maybe later? + # $colfilter[0] = @$filters[6] if ($column =~ /ccode/ ) ; + # $colfilter[0] = @$filters[7] if ($column =~ /location/ ) ; + $colfilter[0] = @$filters[8] if ($column =~ /category/ ) ; + # This commented out row (sort2) was not removed when adding new filters for ccode, shelving location and call number + # $colfilter[0] = @$filters[11] if ($column =~ /sort2/ ) ; $colfilter[0] = @$filters[9] if ($column =~ /timestamp/ ) ; + $colfilter[0] = @$filters[10] if ($column =~ /timestamp/ ) ; + $colfilter[0] = @$filters[11] if ($column =~ /timestamp/ ) ; #warn "filtre col ".$colfilter[0]." ".$colfilter[1]; # loop cols. if ($column eq "Day") { #Display by day - $column = "issues.timestamp"; + $column = "old_issues.timestamp"; $colfield .="dayname($column)"; $colorder .="weekday($column)"; } elsif ($column eq "Month") { #Display by Month - $column = "issues.timestamp"; + $column = "old_issues.timestamp"; $colfield .="monthname($column)"; $colorder .="month($column)"; } elsif ($column eq "Year") { #Display by Year - $column = "issues.timestamp"; + $column = "old_issues.timestamp"; $colfield .="Year($column)"; $colorder .= $column; } else { @@ -269,7 +240,12 @@ sub calculate { } my $strsth2; - $strsth2 .= "select distinctrow $colfield FROM `issues`,borrowers,biblioitems LEFT JOIN items ON (biblioitems.biblioitemnumber=items.biblioitemnumber) WHERE issues.itemnumber=items.itemnumber AND issues.borrowernumber=borrowers.borrowernumber and returndate is not null"; + $strsth2 .= "SELECT distinctrow $colfield + FROM `old_issues` + LEFT JOIN borrowers ON borrowers.borrowernumber=old_issues.borrowernumber + LEFT JOIN items ON old_issues.itemnumber=items.itemnumber + LEFT JOIN biblioitems ON biblioitems.biblioitemnumber=items.biblioitemnumber + WHERE 1"; if (($column=~/timestamp/) or ($column=~/returndate/)){ if ($colfilter[1] and ($colfilter[0])){ $strsth2 .= " and $column between '$colfilter[0]' and '$colfilter[1]' " ; @@ -284,7 +260,6 @@ sub calculate { } $strsth2 .=" group by $colfield"; $strsth2 .=" order by $colorder"; - warn "". $strsth2; my $sth2 = $dbh->prepare( $strsth2 ); if (( @colfilter ) and ($colfilter[1])){ @@ -325,54 +300,66 @@ sub calculate { # Processing average loanperiods $strcalc .= "SELECT DISTINCT biblio.title, COUNT(biblio.biblionumber) AS RANK, biblio.biblionumber AS ID"; + $strcalc .= ", itemcallnumber as CALLNUM"; + $strcalc .= ", ccode as CCODE"; + $strcalc .= ", location as LOC"; $strcalc .= " , $colfield " if ($colfield); - $strcalc .= " FROM `issues`,borrowers,(items LEFT JOIN biblioitems ON biblioitems.biblioitemnumber=items.biblioitemnumber) LEFT JOIN biblio ON (biblio.biblionumber=items.biblionumber) WHERE issues.itemnumber=items.itemnumber AND issues.borrowernumber=borrowers.borrowernumber and returndate is not null"; + $strcalc .= " FROM `old_issues` + LEFT JOIN items USING(itemnumber) + LEFT JOIN biblio USING(biblionumber) + LEFT JOIN biblioitems USING(biblionumber) + LEFT JOIN borrowers USING(borrowernumber) + WHERE 1"; @$filters[0]=~ s/\*/%/g if (@$filters[0]); - $strcalc .= " AND issues.timestamp > '" . @$filters[0] ."'" if ( @$filters[0] ); + $strcalc .= " AND old_issues.timestamp > '" . @$filters[0] ."'" if ( @$filters[0] ); @$filters[1]=~ s/\*/%/g if (@$filters[1]); - $strcalc .= " AND issues.timestamp < '" . @$filters[1] ."'" if ( @$filters[1] ); + $strcalc .= " AND old_issues.timestamp < '" . @$filters[1] ."'" if ( @$filters[1] ); @$filters[2]=~ s/\*/%/g if (@$filters[2]); - $strcalc .= " AND issues.returndate > '" . @$filters[2] ."'" if ( @$filters[2] ); + $strcalc .= " AND old_issues.returndate > '" . @$filters[2] ."'" if ( @$filters[2] ); @$filters[3]=~ s/\*/%/g if (@$filters[3]); - $strcalc .= " AND issues.returndate < '" . @$filters[3] ."'" if ( @$filters[3] ); + $strcalc .= " AND old_issues.returndate < '" . @$filters[3] ."'" if ( @$filters[3] ); @$filters[4]=~ s/\*/%/g if (@$filters[4]); - $strcalc .= " AND issues.branchcode like '" . @$filters[4] ."'" if ( @$filters[4] ); + $strcalc .= " AND old_issues.branchcode like '" . @$filters[4] ."'" if ( @$filters[4] ); @$filters[5]=~ s/\*/%/g if (@$filters[5]); - $strcalc .= " AND biblioitems.itemtype like '" . @$filters[5] ."'" if ( @$filters[5] ); + if ( @$filters[5] ){ + if(C4::Context->preference('item-level_itypes') ){ + $strcalc .= " AND items.itype like " + }else{ + $strcalc .= " AND biblioitems.itemtype like " + } + $strcalc .= "'" . @$filters[5] ."'" ; + } @$filters[6]=~ s/\*/%/g if (@$filters[6]); - $strcalc .= " AND borrowers.categorycode like '" . @$filters[6] ."'" if ( @$filters[6] ); + $strcalc .= " AND itemcallnumber like '" . @$filters[6] ."'" if ( @$filters[6] ); @$filters[7]=~ s/\*/%/g if (@$filters[7]); - $strcalc .= " AND dayname(issues.timestamp) like '" . @$filters[7]."'" if (@$filters[7]); + $strcalc .= " AND ccode like '" . @$filters[7] ."'" if ( @$filters[7] ); @$filters[8]=~ s/\*/%/g if (@$filters[8]); - $strcalc .= " AND monthname(issues.timestamp) like '" . @$filters[8]."'" if (@$filters[8]); + $strcalc .= " AND location like '" . @$filters[8] ."'" if ( @$filters[8] ); @$filters[9]=~ s/\*/%/g if (@$filters[9]); - $strcalc .= " AND year(issues.timestamp) like '" . @$filters[9] ."'" if ( @$filters[9] ); + $strcalc .= " AND borrowers.categorycode like '" . @$filters[9] ."'" if ( @$filters[9] ); + @$filters[10]=~ s/\*/%/g if (@$filters[10]); + $strcalc .= " AND dayname(old_issues.timestamp) like '" . @$filters[10]."'" if (@$filters[10]); + @$filters[11]=~ s/\*/%/g if (@$filters[11]); + $strcalc .= " AND monthname(old_issues.timestamp) like '" . @$filters[11]."'" if (@$filters[11]); + @$filters[12]=~ s/\*/%/g if (@$filters[12]); + $strcalc .= " AND year(old_issues.timestamp) like '" . @$filters[12] ."'" if ( @$filters[12] ); $strcalc .= " group by biblio.biblionumber"; $strcalc .= ", $colfield" if ($column); $strcalc .= " order by RANK DESC"; $strcalc .= ", $colfield " if ($colfield); -# my $max; -# if (@loopcol) { -# $max = $line*@loopcol; -# } else { $max=$line;} -# $strcalc .= " LIMIT 0,$max"; - warn "SQL :". $strcalc; my $dbcalc = $dbh->prepare($strcalc); $dbcalc->execute; -# warn "filling table"; - my $previous_col; my %indice; while (my @data = $dbcalc->fetchrow) { - my ($row, $rank, $id, $col )=@data; - $col = "zzEMPTY" if ($col eq undef); + my ($row, $rank, $id, $callnum, $ccode, $loc, $col )=@data; + $col = "zzEMPTY" if (!defined($col)); $indice{$col}=1 if (not($indice{$col})); $table[$indice{$col}]->{$col}->{'name'}=$row; $table[$indice{$col}]->{$col}->{'count'}=$rank; $table[$indice{$col}]->{$col}->{'link'}=$id; -# warn " ".$i." ".$col. " ".$row; $indice{$col}++; } @@ -380,12 +367,10 @@ sub calculate { for ($i=1; $i<=$line;$i++) { my @loopcell; - warn " $i"; #@loopcol ensures the order for columns is common with column titles # and the number matches the number of columns my $colcount=0; foreach my $col ( @loopcol ) { -# warn " colonne :$col->{coltitle}"; my $value; my $count=0; my $link; @@ -398,15 +383,12 @@ sub calculate { $count =$table[$i]->{"zzEMPTY"}->{'count'}; $link =$table[$i]->{"zzEMPTY"}->{'link'}; } -# warn " ".$i ." value:$value count:$count reference:$link"; push @loopcell, {value => $value, count =>$count, reference => $link} ; } - #warn "row : $row colcount:$colcount"; #my $total = $table[$i]->{totalrow}/$colcount if ($colcount>0); push @looprow,{ 'rowtitle' => $i , 'loopcell' => \@loopcell, 'hilighted' => ($hilighted >0), - #'totalrow' => ($total)?sprintf("%.2f",$total):0 }; $hilighted = -$hilighted; } @@ -419,7 +401,6 @@ sub calculate { $globalline{looprow} = \@looprow; $globalline{loopcol} = \@loopcol; # # the foot (totals by borrower type) - $globalline{loopfooter} = \@loopfooter; $globalline{total}= $grantotal; $globalline{line} = $line; $globalline{column} = $column;