Bug 7113: Standardize vendor id name in templates and scripts

[koha.git] / acqui / lateorders.pl

diff --git a/acqui/lateorders.pl b/acqui/lateorders.pl
index 720dd8e..86b1300 100755 (executable)
--- a/acqui/lateorders.pl
+++ b/acqui/lateorders.pl
@@ -15,79 +15,114 @@
 # Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
 # Suite 330, Boston, MA  02111-1307 USA
 
-# $Id$
+
+=head1 NAME
+
+lateorders.pl
+
+=head1 DESCRIPTION
+
+this script shows late orders for a specific supplier, branch and delay
+given on input arg.
+
+=head1 CGI PARAMETERS
+
+=over 4
+
+=item booksellerid
+To know on which supplier this script have to display late order.
+
+=item delay
+To know the time boundary. Default value is 30 days.
+
+=item branch
+To know on which branch this script have to display late order.
+
+=back
+
+=cut
 
 use strict;
+use warnings;
 use CGI;
-use C4::Acquisition;
+use C4::Bookseller qw( GetBooksellersWithLateOrders );
 use C4::Auth;
 use C4::Koha;
 use C4::Output;
-use C4::Interface::CGI::Output;
 use C4::Context;
-use HTML::Template;
-
-my $query = new CGI;
-my ($template, $loggedinuser, $cookie)
-= get_template_and_user({template_name => "acqui/lateorders.tmpl",
-                               query => $query,
-                               type => "intranet",
-                               authnotrequired => 0,
-                               flagsrequired => {acquisition => 1},
-                               debug => 1,
-                               });
-# my $title = $query->param('title');
-# my $ISSN = $query->param('ISSN');
-# my @subscriptions = GetSubscriptions($title,$ISSN);
-
-my $supplierid = $query->param('supplierid');
-my $delay = $query->param('delay');
-my $branch = $query->param('branch');
-
-$delay =($delay?$delay:30);
-
-my %supplierlist = getsupplierlistwithlateorders($delay,$branch);
-my @select_supplier;
-push @select_supplier,"";
-foreach my $supplierid (keys %supplierlist){
-       push @select_supplier, $supplierid;
+use C4::Acquisition;
+use C4::Letters;
+use C4::Branch; # GetBranches
+
+my $input = new CGI;
+my ($template, $loggedinuser, $cookie) = get_template_and_user({
+       template_name => "acqui/lateorders.tmpl",
+       query => $input,
+        type => "intranet",
+       authnotrequired => 0,
+       flagsrequired => {acquisition => 'order_receive'},
+       debug => 1,
+});
+
+my $booksellerid = $input->param('booksellerid') || undef; # we don't want "" or 0
+my $delay      = $input->param('delay');
+my $branch     = $input->param('branch');
+my $op         = $input->param('op');
+
+my @errors = ();
+$delay = 30 unless defined $delay;
+unless ($delay =~ /^\d{1,3}$/) {
+       push @errors, {delay_digits => 1, bad_delay => $delay};
+       $delay = 30;    #default value for delay
 }
-my $CGIsupplier=CGI::scrolling_list( -name     => 'supplierid',
-                       -values   => \@select_supplier,
-                       -default  => $supplierid,
-                       -labels   => \%supplierlist,
-                       -size     => 1,
-                       -tabindex=>'',
-                       -multiple => 0 );
-
-$template->param(Supplier=>$supplierlist{$supplierid}) if ($supplierid);
-
-my $branches = getbranches;
-my @branchloop;
-foreach my $thisbranch (sort keys %$branches) {
-       my %row =(value => $thisbranch,
-                               branchname => $branches->{$thisbranch}->{'branchname'},
-                       );
-       push @branchloop, \%row;
+
+if ($op and $op eq "send_alert"){
+    my @ordernums = $input->param("claim_for");# FIXME: Fallback values?
+    my $err;
+    eval {
+        $err = SendAlerts( 'claimacquisition', \@ordernums, $input->param("letter_code") );    # FIXME: Fallback value?
+        AddClaim ( $_ ) for @ordernums;
+    };
+    if ( $@ ) {
+        $template->param(error_claim => $@);
+    } elsif ( defined $err->{error} and $err->{error} eq "no_email" ) {
+        $template->{VARS}->{'error_claim'} = "no_email";
+    } else {
+        $template->{VARS}->{'info_claim'} = 1;
+    }
+}
+
+my %supplierlist = GetBooksellersWithLateOrders($delay);
+my (@sloopy);  # supplier loop
+foreach (keys %supplierlist){
+       push @sloopy, (($booksellerid and $booksellerid eq $_ )            ?
+                                       {id=>$_, name=>$supplierlist{$_}, selected=>1} :
+                                       {id=>$_, name=>$supplierlist{$_}} )            ;
 }
-my $CGIbranch=CGI::scrolling_list( -name     => 'branch',
-                               -values   => \@select_branches,
-                               -labels   => \%select_branches,
-                               -size     => 1,
-                               -tabindex=>'',
-                               -multiple => 0 );
-
-my ($count, @lateorders) = getlateorders($delay,$supplierid,$branch);
+$template->param(SUPPLIER_LOOP => \@sloopy);
+
+$template->param(Supplier=>$supplierlist{$booksellerid}) if ($booksellerid);
+$template->param(booksellerid=>$booksellerid) if ($booksellerid);
+
+my @lateorders = GetLateOrders($delay,$booksellerid,$branch);
+
 my $total;
-foreach my $lateorder (@lateorders){
-       $total+=$lateorder->{subtotal};
+foreach (@lateorders){
+       $total += $_->{subtotal};
 }
-$template->param(delay=>$delay) if ($delay);
+
+my @letters;
+my $letters=GetLetters("claimacquisition");
+foreach (keys %$letters){
+       push @letters, {code=>$_,name=>$letters->{$_}};
+}
+$template->param(letters=>\@letters) if (@letters);
+
+$template->param(ERROR_LOOP => \@errors) if (@errors);
 $template->param(
-       branchloop => \@branchloop,
-       CGIsupplier => $CGIsupplier,
        lateorders => \@lateorders,
-       total=>$total,
+       delay => $delay,
+       total => $total,
        intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"),
-       );
-output_html_with_http_headers $query, $cookie, $template->output;
+);
+output_html_with_http_headers $input, $cookie, $template->output;