Jonathan Druart
62285d2de4
The idea of output_and_exit_if_error (added by bug 18403) is to make sure parameters are valid before executing the script. If not (old or broken URLs), we shortcut everything coming next to display a generic error ("object does not exist", "you do not have permission to do that", etc.) This bug report fixes the scripts under serials/*. Test plan: Hit the script under the serials directory with an invalid subscriptionid parameter and confirm you get an error instead of the normal view with empty values. The goal is not to be exhaustive during the first iteration, but at least to fix the most common views. For instance: /cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=XXX /cgi-bin/koha/serials/serials-collection.pl?subscriptionid=XXX /cgi-bin/koha/serials/routing.pl?subscriptionid=XXX&op=new /cgi-bin/koha/serials/subscription-add.pl?op=modify&subscriptionid=XXx /cgi-bin/koha/serials/subscription-add.pl?op=dup&subscriptionid=XXX Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
187 lines
7.7 KiB
Perl
Executable file
187 lines
7.7 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
# Copyright 2000-2002 Katipo Communications
|
|
# Parts Copyright 2010 Biblibre
|
|
#
|
|
# 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 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.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with Koha; if not, see <http://www.gnu.org/licenses>.
|
|
|
|
|
|
use Modern::Perl;
|
|
use CGI qw ( -utf8 );
|
|
use C4::Auth;
|
|
use C4::Koha;
|
|
use C4::Serials;
|
|
use C4::Letters;
|
|
use C4::Output;
|
|
use C4::Context;
|
|
|
|
use Koha::DateUtils qw( dt_from_string );
|
|
|
|
use List::MoreUtils qw/uniq/;
|
|
|
|
|
|
my $query = new CGI;
|
|
my $op = $query->param('op') || q{};
|
|
my $nbissues=$query->param('nbissues');
|
|
my $date_received_today = $query->param('date_received_today') || 0;
|
|
my $dbh = C4::Context->dbh;
|
|
|
|
my ($template, $loggedinuser, $cookie)
|
|
= get_template_and_user({template_name => "serials/serials-collection.tt",
|
|
query => $query,
|
|
type => "intranet",
|
|
authnotrequired => 0,
|
|
flagsrequired => {serials => '*'},
|
|
debug => 1,
|
|
});
|
|
my $biblionumber = $query->param('biblionumber');
|
|
my @subscriptionid = $query->multi_param('subscriptionid');
|
|
|
|
@subscriptionid= uniq @subscriptionid;
|
|
@subscriptionid= sort @subscriptionid;
|
|
my $subscriptiondescs;
|
|
my $subscriptions;
|
|
|
|
if($op eq 'gennext' && @subscriptionid){
|
|
my $subscriptionid = $subscriptionid[0];
|
|
my $sth = $dbh->prepare("
|
|
SELECT publisheddate, publisheddatetext, serialid, serialseq,
|
|
planneddate
|
|
FROM serial
|
|
WHERE status = 1 AND subscriptionid = ?
|
|
");
|
|
my $status = defined( $nbissues ) ? 2 : 3;
|
|
$nbissues ||= 1;
|
|
for ( my $i = 0; $i < $nbissues; $i++ ){
|
|
$sth->execute($subscriptionid);
|
|
# modify actual expected issue, to generate the next
|
|
if ( my $issue = $sth->fetchrow_hashref ) {
|
|
my $planneddate = $date_received_today ? dt_from_string : $issue->{planneddate};
|
|
ModSerialStatus( $issue->{serialid}, $issue->{serialseq},
|
|
$planneddate, $issue->{publisheddate},
|
|
$issue->{publisheddatetext}, $status, "" );
|
|
} else {
|
|
require C4::Serials::Numberpattern;
|
|
my $subscription = GetSubscription($subscriptionid);
|
|
my $pattern = C4::Serials::Numberpattern::GetSubscriptionNumberpattern($subscription->{numberpattern});
|
|
my $expected = GetNextExpected($subscriptionid);
|
|
my (
|
|
$newserialseq, $newlastvalue1, $newlastvalue2, $newlastvalue3,
|
|
$newinnerloop1, $newinnerloop2, $newinnerloop3
|
|
) = GetNextSeq($subscription, $pattern, $expected->{publisheddate});
|
|
|
|
## We generate the next publication date
|
|
my $nextpublisheddate = GetNextDate($subscription, $expected->{publisheddate}, 1);
|
|
my $planneddate = $date_received_today ? dt_from_string : $nextpublisheddate;
|
|
## Creating the new issue
|
|
NewIssue( $newserialseq, $subscriptionid, $subscription->{'biblionumber'},
|
|
1, $planneddate, $nextpublisheddate );
|
|
|
|
## Updating the subscription seq status
|
|
my $squery = "UPDATE subscription SET lastvalue1=?, lastvalue2=?, lastvalue3=?, innerloop1=?, innerloop2=?, innerloop3=?
|
|
WHERE subscriptionid = ?";
|
|
my $seqsth = $dbh->prepare($squery);
|
|
$seqsth->execute(
|
|
$newlastvalue1, $newlastvalue2, $newlastvalue3, $newinnerloop1,
|
|
$newinnerloop2, $newinnerloop3, $subscriptionid
|
|
);
|
|
|
|
}
|
|
last if $nbissues == 1;
|
|
last if HasSubscriptionExpired($subscriptionid) > 0;
|
|
}
|
|
print $query->redirect('/cgi-bin/koha/serials/serials-collection.pl?subscriptionid='.$subscriptionid);
|
|
exit;
|
|
}
|
|
|
|
my $subscriptioncount;
|
|
my ($location, $callnumber);
|
|
if (@subscriptionid){
|
|
my @subscriptioninformation=();
|
|
my $closed = 0;
|
|
foreach my $subscriptionid (@subscriptionid){
|
|
my $subs= GetSubscription($subscriptionid);
|
|
next unless $subs;
|
|
$closed = 1 if $subs->{closed};
|
|
|
|
$subs->{opacnote} =~ s/\n/\<br\/\>/g;
|
|
$subs->{missinglist} =~ s/\n/\<br\/\>/g;
|
|
$subs->{recievedlist} =~ s/\n/\<br\/\>/g;
|
|
|
|
##these are display information
|
|
$subs->{'abouttoexpire'}=abouttoexpire($subs->{'subscriptionid'});
|
|
$subs->{'subscriptionexpired'}=HasSubscriptionExpired($subs->{'subscriptionid'});
|
|
$subs->{'subscriptionid'} = $subscriptionid; # FIXME - why was this lost ?
|
|
$location = $subs->{'location'};
|
|
$callnumber = $subs->{callnumber};
|
|
my $frequency = C4::Serials::Frequency::GetSubscriptionFrequency($subs->{periodicity});
|
|
my $numberpattern = C4::Serials::Numberpattern::GetSubscriptionNumberpattern($subs->{numberpattern});
|
|
$subs->{frequency} = $frequency;
|
|
$subs->{numberpattern} = $numberpattern;
|
|
$subs->{'hasRouting'} = check_routing($subscriptionid);
|
|
push @$subscriptiondescs,$subs;
|
|
my $tmpsubscription= GetFullSubscription($subscriptionid);
|
|
@subscriptioninformation=(@$tmpsubscription,@subscriptioninformation);
|
|
}
|
|
|
|
output_and_exit( $query, $cookie, $template, 'unknown_subscription') unless @subscriptioninformation;
|
|
|
|
$template->param(closed => $closed);
|
|
$subscriptions=PrepareSerialsData(\@subscriptioninformation);
|
|
$subscriptioncount = CountSubscriptionFromBiblionumber($subscriptiondescs->[0]{'biblionumber'});
|
|
} else {
|
|
$subscriptiondescs = GetSubscriptionsFromBiblionumber($biblionumber);
|
|
foreach my $s (@$subscriptiondescs) {
|
|
my $frequency = C4::Serials::Frequency::GetSubscriptionFrequency($s->{periodicity});
|
|
my $numberpattern = C4::Serials::Numberpattern::GetSubscriptionNumberpattern($s->{numberpattern});
|
|
$s->{frequency} = $frequency;
|
|
$s->{numberpattern} = $numberpattern;
|
|
}
|
|
my $subscriptioninformation = GetFullSubscriptionsFromBiblionumber($biblionumber);
|
|
$subscriptions=PrepareSerialsData($subscriptioninformation);
|
|
}
|
|
|
|
my $title = $subscriptiondescs->[0]{bibliotitle};
|
|
my $yearmax=($subscriptions->[0]{year} eq "manage" && scalar(@$subscriptions)>1)? $subscriptions->[1]{year} :$subscriptions->[0]{year};
|
|
my $yearmin=$subscriptions->[scalar(@$subscriptions)-1]{year};
|
|
my $subscriptionidlist="";
|
|
foreach my $subscription (@$subscriptiondescs){
|
|
$subscriptionidlist.=$subscription->{'subscriptionid'}."," ;
|
|
$biblionumber = $subscription->{'bibnum'} unless ($biblionumber);
|
|
$subscription->{'hasRouting'} = check_routing($subscription->{'subscriptionid'});
|
|
}
|
|
|
|
chop $subscriptionidlist;
|
|
$template->param(
|
|
subscriptionidlist => $subscriptionidlist,
|
|
biblionumber => $biblionumber,
|
|
subscriptions => $subscriptiondescs,
|
|
years => $subscriptions,
|
|
yearmin => $yearmin,
|
|
yearmax =>$yearmax,
|
|
bibliotitle => $title,
|
|
suggestion => C4::Context->preference("suggestion"),
|
|
virtualshelves => C4::Context->preference("virtualshelves"),
|
|
routing => C4::Context->preference("RoutingSerials"),
|
|
subscr=>scalar $query->param('subscriptionid'),
|
|
subscriptioncount => $subscriptioncount,
|
|
location => $location,
|
|
callnumber => $callnumber,
|
|
uc(C4::Context->preference("marcflavour")) => 1,
|
|
serialsadditems => $subscriptiondescs->[0]{'serialsadditems'},
|
|
);
|
|
|
|
output_html_with_http_headers $query, $cookie, $template->output;
|