my %supplierlist = GetSuppliersWithLateIssues;
my @select_supplier;
-foreach my $supplierid (keys %supplierlist){
+# open template first (security & userenv set here)
+my ($template, $loggedinuser, $cookie)
+= get_template_and_user({template_name => "serials/claims.tmpl",
+ query => $input,
+ type => "intranet",
+ authnotrequired => 0,
+ flagsrequired => {serials => 1},
+ debug => 1,
+ });
+
+foreach my $supplierid (sort {$supplierlist{$a} cmp $supplierlist{$b} } keys %supplierlist){
my ($count, @dummy) = GetLateOrMissingIssues($supplierid,"",$order);
my $counting = $count;
$supplierlist{$supplierid} = $supplierlist{$supplierid}." ($counting)";
push @letters ,{code=>$_,name=> $letters->{$_}};
}
-my $letter=((scalar(@letters)>1)||($letters[0]->{name}||$letters[0]->{code}));
+my $letter=((scalar(@letters)>1) || ($letters[0]->{name}||$letters[0]->{code}));
my ($count2, @missingissues) = GetLateOrMissingIssues($supplierid,$serialid,$order) if $supplierid;
my $CGIsupplier=CGI::scrolling_list( -name => 'supplierid',
+ -id => 'supplierid',
-values => \@select_supplier,
-default => $supplierid,
-labels => \%supplierlist,
-size => 1,
- -multiple => 0
- -onChange => 'onchange="submit();"');
+ -multiple => 0 );
my ($singlesupplier,@supplierinfo);
if($supplierid){
}
}
-
my $preview=0;
if($op eq 'preview'){
$preview = 1;
+} else {
+ my @serialnums=$input->param('serialid');
+ if (@serialnums) { # i.e. they have been flagged to generate claims
+ SendAlerts('claimissues',\@serialnums,$input->param("letter_code"));
+ my $cntupdate=UpdateClaimdateIssues(\@serialnums);
+ ### $cntupdate SHOULD be equal to scalar(@$serialnums) TODO so what do we do about it??
+ }
}
-if ($op eq "send_alert"){
- my @serialnums=$input->param("serialid");
- SendAlerts('claimissues',\@serialnums,$input->param("letter_code"));
- my $cntupdate=UpdateClaimdateIssues(\@serialnums);
- ### $cntupdate SHOULD be equal to scalar(@$serialnums)
-}
-
-my ($template, $loggedinuser, $cookie)
-= get_template_and_user({template_name => "serials/claims.tmpl",
- query => $input,
- type => "intranet",
- authnotrequired => 0,
- flagsrequired => {serials => 1},
- debug => 1,
- });
-
$template->param('letters'=>\@letters,'letter'=>$letter);
$template->param(
- order =>$order,
- CGIsupplier => $CGIsupplier,
- phone => $supplierinfo[0]->{phone},
- booksellerfax => $supplierinfo[0]->{booksellerfax},
- bookselleremail => $supplierinfo[0]->{bookselleremail},
+ order =>$order,
+ CGIsupplier => $CGIsupplier,
+ phone => $supplierinfo[0]->{phone},
+ booksellerfax => $supplierinfo[0]->{booksellerfax},
+ bookselleremail => $supplierinfo[0]->{bookselleremail},
preview => $preview,
missingissues => \@missingissues,
supplierid => $supplierid,
claimletter => $claimletter,
singlesupplier => $singlesupplier,
supplierloop => \@supplierinfo,
- intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"),
- intranetstylesheet => C4::Context->preference("intranetstylesheet"),
- IntranetNav => C4::Context->preference("IntranetNav"),
- );
+ dateformat => C4::Context->preference("dateformat"),
+ );
output_html_with_http_headers $input, $cookie, $template->output;