X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=serials%2Fsubscription-renew.pl;h=f92ee37bf402dd138427509917aebdb282db7168;hb=fbe215e6c6ecf1f4c45ed152a733fe6e77cc0e15;hp=ab6a01991209d1d94474cf28b979ef2c4834bbc4;hpb=d71a9d3d5b0405eabb3213fd2c4962a74283aa09;p=koha_gimpoz

diff --git a/serials/subscription-renew.pl b/serials/subscription-renew.pl
index ab6a019912..f92ee37bf4 100755
--- a/serials/subscription-renew.pl
+++ b/serials/subscription-renew.pl
@@ -1,5 +1,4 @@
 #!/usr/bin/perl
-# WARNING: 4-character tab stops here
 
 # Copyright 2000-2002 Katipo Communications
 #
@@ -18,7 +17,6 @@
 # Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
 # Suite 330, Boston, MA  02111-1307 USA
 
-# $Id$
 
 =head1 NAME
 
@@ -46,12 +44,12 @@ Id of the subscription this script has to renew
 =cut
 
 use strict;
-require Exporter;
+use warnings;
+
 use CGI;
 use C4::Koha;
 use C4::Auth;
-use C4::Date;
-
+use C4::Dates qw/format_date/;
 use C4::Context;
 use C4::Auth;
 use C4::Output;
@@ -60,7 +58,8 @@ use C4::Serials;
 my $query = new CGI;
 my $dbh   = C4::Context->dbh;
 
-my $op             = $query->param('op');
+my $mode           = $query->param('mode');
+my $op             = $query->param('op') || q{};
 my $subscriptionid = $query->param('subscriptionid');
 my $done = 0;    # for after form has been submitted
 my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
@@ -81,11 +80,13 @@ if ( $op eq "renew" ) {
         $query->param('weeklength'), $query->param('monthlength'),
         $query->param('note')
     );
-     print $query->redirect('/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid='.$subscriptionid);
-     exit;
 }
 
 my $subscription = GetSubscription($subscriptionid);
+if ($subscription->{'cannotedit'}){
+  warn "Attempt to renew subscription $subscriptionid by ".C4::Context->userenv->{'id'}." not allowed";
+  print $query->redirect("/cgi-bin/koha/serials/subscription-detail.pl?subscriptionid=$subscriptionid");
+}
 
 $template->param(
     startdate => format_date(
@@ -98,12 +99,8 @@ $template->param(
     subscriptionid => $subscriptionid,
     bibliotitle    => $subscription->{bibliotitle},
     $op            => 1,
-    done           => $done,
+    popup          => ($query->param('mode')eq "popup"),
 );
 
 # Print the page
 output_html_with_http_headers $query, $cookie, $template->output;
-
-# Local Variables:
-# tab-width: 4
-# End: