use Modern::Perl;
use CGI qw ( -utf8 );
-use Carp;
+use Carp qw( carp );
use C4::Koha;
-use C4::Auth;
+use C4::Auth qw( get_template_and_user );
use C4::Context;
-use C4::Auth;
-use C4::Output;
-use C4::Serials;
-use Koha::DateUtils;
+use C4::Auth qw( get_template_and_user );
+use C4::Output qw( output_and_exit output_html_with_http_headers );
+use C4::Serials qw( GetSubscription GetSubscriptionLength NewSubscription ReNewSubscription );
+use Koha::DateUtils qw( dt_from_string output_pref );
-my $query = new CGI;
+my $query = CGI->new;
my $dbh = C4::Context->dbh;
my $mode = $query->param('mode') || q{};
my $op = $query->param('op') || 'display';
-my $subscriptionid = $query->param('subscriptionid');
+my @subscriptionids = $query->multi_param('subscriptionid');
+my $branchcode = $query->param('branchcode');
+my $sublength = $query->param('sublength');
+my $subtype = $query->param('subtype');
+my ($numberlength, $weeklength, $monthlength);
+
my $done = 0; # for after form has been submitted
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{
template_name => "serials/subscription-renew.tt",
query => $query,
type => "intranet",
- authnotrequired => 0,
flagsrequired => { serials => 'renew_subscription' },
- debug => 1,
}
);
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' } );
+ ($numberlength, $weeklength, $monthlength) = GetSubscriptionLength( $subtype, $sublength );
ReNewSubscription(
- $subscriptionid, $loggedinuser,
- $startdate, $query->param('numberlength'),
- $query->param('weeklength'), $query->param('monthlength'),
- $query->param('note')
+ {
+ subscriptionid => $subscriptionid,
+ user => $loggedinuser,
+ startdate => $startdate,
+ numberlength => $numberlength,
+ weeklength => $weeklength,
+ monthlength => $monthlength,
+ note => scalar $query->param('note'),
+ branchcode => $branchcode
+ }
);
-}
+} elsif ( $op eq 'multi_renew' ) {
+ for my $subscriptionid ( @subscriptionids ) {
+ my $subscription = GetSubscription( $subscriptionid );
+ next unless $subscription;
+ ReNewSubscription(
+ {
+ subscriptionid => $subscriptionid,
+ user => $loggedinuser,
+ startdate => $subscription->{enddate},
+ numberlength => $subscription->{numberlength},
+ weeklength => $subscription->{weeklength},
+ monthlength => $subscription->{monthlength},
+ }
+ );
+ }
+} 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");
+ }
-my $subscription = GetSubscription($subscriptionid);
-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");
-}
+ my $newstartdate = output_pref( { str => $subscription->{enddate}, dateonly => 1 } )
+ or output_pref( { dt => dt_from_string, dateonly => 1 } );
-my $newstartdate = output_pref( { str => $subscription->{enddate}, dateonly => 1 } )
- or output_pref( { dt => dt_from_string, dateonly => 1 } );
+ $template->param(
+ startdate => $newstartdate,
+ subscription => $subscription,
+ );
+}
$template->param(
- startdate => $newstartdate,
- numberlength => $subscription->{numberlength},
- weeklength => $subscription->{weeklength},
- monthlength => $subscription->{monthlength},
- subscriptionid => $subscriptionid,
- bibliotitle => $subscription->{bibliotitle},
- $op => 1,
- popup => ($mode eq 'popup'),
+ op => $op,
);
-# Print the page
output_html_with_http_headers $query, $cookie, $template->output;