Bug 5549 : Fix a couple of date issues from testing
[koha.git] / acqui / lateorders.pl
index 2482ffb..86b1300 100755 (executable)
@@ -29,7 +29,7 @@ given on input arg.
 
 =over 4
 
-=item supplierid
+=item booksellerid
 To know on which supplier this script have to display late order.
 
 =item delay
@@ -64,7 +64,7 @@ my ($template, $loggedinuser, $cookie) = get_template_and_user({
        debug => 1,
 });
 
-my $supplierid = $input->param('supplierid') || undef; # we don't want "" or 0
+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');
@@ -78,29 +78,33 @@ unless ($delay =~ /^\d{1,3}$/) {
 
 if ($op and $op eq "send_alert"){
     my @ordernums = $input->param("claim_for");# FIXME: Fallback values?
+    my $err;
     eval {
-        SendAlerts( 'claimacquisition', \@ordernums, $input->param("letter_code") );    # FIXME: Fallback value?
+        $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->param(info_claim => "Emails have been sent");
+        $template->{VARS}->{'info_claim'} = 1;
     }
 }
 
 my %supplierlist = GetBooksellersWithLateOrders($delay);
 my (@sloopy);  # supplier loop
 foreach (keys %supplierlist){
-       push @sloopy, (($supplierid and $supplierid eq $_ )            ? 
+       push @sloopy, (($booksellerid and $booksellerid eq $_ )            ?
                                        {id=>$_, name=>$supplierlist{$_}, selected=>1} :
                                        {id=>$_, name=>$supplierlist{$_}} )            ;
 }
 $template->param(SUPPLIER_LOOP => \@sloopy);
-$template->param(Supplier=>$supplierlist{$supplierid}) if ($supplierid);
-$template->param(SupplierId=>$supplierid) if ($supplierid);
 
-my @lateorders = GetLateOrders($delay,$supplierid,$branch);
+$template->param(Supplier=>$supplierlist{$booksellerid}) if ($booksellerid);
+$template->param(booksellerid=>$booksellerid) if ($booksellerid);
+
+my @lateorders = GetLateOrders($delay,$booksellerid,$branch);
 
 my $total;
 foreach (@lateorders){