$sth->execute($subscriptionid);
my $subscription = $sth->fetchrow_hashref;
+ return unless $subscription;
+
$subscription->{cannotedit} = not can_edit_subscription( $subscription );
# Add additional fields to the subscription into a new key "additional_fields"
<div class="dialog message">This bibliographic record does not exist.</div>
[% CASE 'unknown_item' %]
<div class="dialog message">This item does not exist.</div>
+ [% CASE 'unknown_subscription' %]
+ <div class="dialog message">This subscription does not exist.</div>
[% CASE %][% blocking_error | html %]
[% END %]
[% INCLUDE 'header.inc' %]
[% INCLUDE 'serials-search.inc' %]
-<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/serials/serials-home.pl">Serials</a> › <a href="/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=[% subscriptionid | html %]"><i>[% title | html %]</i></a> › [% IF ( op ) %]Create routing list[% ELSE %]Edit routing list[% END %]</div>
+<div id="breadcrumbs">
+ <a href="/cgi-bin/koha/mainpage.pl">Home</a>
+ › <a href="/cgi-bin/koha/serials/serials-home.pl">Serials</a>
+ [% UNLESS blocking_error %]
+ › <a href="/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=[% subscriptionid | uri %]"><i>[% title | html %]</i></a>
+ › [% IF ( op ) %]Create routing list[% ELSE %]Edit routing list[% END %]
+ [% END %]
+</div>
<div id="doc3" class="yui-t2">
<div id="yui-main">
<div class="yui-b">
+[% INCLUDE 'blocking_errors.inc' %]
[% IF ( op ) %]
<h1>Create routing list for <i>[% title | html %]</i></h1>
[% INCLUDE 'header.inc' %]
[% INCLUDE 'serials-search.inc' %]
-<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/serials/serials-home.pl">Serials</a> › [% IF ( modify ) %]<a href="/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=[% subscriptionid | html %]"><i>[% bibliotitle | html %]</i></a> › Modify subscription[% ELSE %]New subscription[% END %]</div>
+<div id="breadcrumbs">
+ <a href="/cgi-bin/koha/mainpage.pl">Home</a>
+ › <a href="/cgi-bin/koha/serials/serials-home.pl">Serials</a>
+ [% UNLESS blocking_error %]
+ › [% IF ( modify ) %]<a href="/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=[% subscriptionid | uri %]"><i>[% bibliotitle | html %]</i></a> › Modify subscription[% ELSE %]New subscription[% END %]
+ [% END %]
+</div>
+[% INCLUDE 'blocking_errors.inc' %]
<div class="main container-fluid">
<div class="row">
[% INCLUDE 'header.inc' %]
[% INCLUDE 'serials-search.inc' %]
-<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/serials/serials-home.pl">Serials</a> › Details for subscription #[% subscriptionid | html %]</div>
+<div id="breadcrumbs">
+ <a href="/cgi-bin/koha/mainpage.pl">Home</a>
+ › <a href="/cgi-bin/koha/serials/serials-home.pl">Serials</a>
+ [% UNLESS blocking_error %]› Details for subscription #[% subscriptionid | html %][% END %]
+</div>
<div id="doc3" class="yui-t2">
<body id="ser_subscription-renew" class="ser">
<div class="container-fluid">
+[% INCLUDE 'blocking_errors.inc' %]
+
[% IF op == 'renew' OR op =='multi_renew' %]
[% IF op == 'renew' %]
<span>Subscription renewed.<span>
}
);
+my $subs = GetSubscription($subscriptionid);
+
+output_and_exit( $query, $cookie, $template, 'unknown_subscription')
+ unless $subs;
+
if($op eq 'delete'){
delroutingmember($routingid,$subscriptionid);
}
}
my @routinglist = getroutinglist($subscriptionid);
-my $subs = GetSubscription($subscriptionid);
+
my ($count,@serials) = GetSerials($subscriptionid);
my $serialdates = GetLatestSerials($subscriptionid,$count);
last if HasSubscriptionExpired($subscriptionid) > 0;
}
print $query->redirect('/cgi-bin/koha/serials/serials-collection.pl?subscriptionid='.$subscriptionid);
+ exit;
}
my $subscriptioncount;
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;
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'});
$string =~ s/,$//;
print $query->redirect($string);
+ exit;
}
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
my $subscriptionid = $query->param('subscriptionid');
$subs = GetSubscription($subscriptionid);
+ output_and_exit( $query, $cookie, $template, 'unknown_subscription')
+ unless $subs;
+
## FIXME : Check rights to edit if mod. Could/Should display an error message.
if ($subs->{'cannotedit'} && $op eq 'modify'){
carp "Attempt to modify subscription $subscriptionid by ".C4::Context->userenv->{'id'}." not allowed";
my $subs = GetSubscription($subscriptionid);
+
+output_and_exit( $query, $cookie, $template, 'unknown_subscription')
+ unless $subs;
+
$subs->{enddate} ||= GetExpirationDate($subscriptionid);
my ($totalissues,@serialslist) = GetSerials($subscriptionid);
if ( $op eq "renew" ) {
# Do not use this script with op=renew and @subscriptionids > 1!
my $subscriptionid = $subscriptionids[0];
+ # Make sure the subscription exists
+ my $subscription = GetSubscription( $subscriptionid );
+ output_and_exit( $query, $cookie, $template, 'unknown_subscription') unless $subscription;
my $startdate = output_pref( { str => scalar $query->param('startdate'), dateonly => 1, dateformat => 'iso' } );
ReNewSubscription(
$subscriptionid, $loggedinuser,
} else {
my $subscriptionid = $subscriptionids[0];
my $subscription = GetSubscription($subscriptionid);
+ output_and_exit( $query, $cookie, $template, 'unknown_subscription') unless $subscription;
if ($subscription->{'cannotedit'}){
carp "Attempt to renew subscription $subscriptionid by ".C4::Context->userenv->{'id'}." not allowed";
print $query->redirect("/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=$subscriptionid");