Bug 5549 : Duedate formatting : reserves and opac

[koha.git] / reserve / renewscript.pl

diff --git a/reserve/renewscript.pl b/reserve/renewscript.pl
index 80d2750..4b80d64 100755 (executable)
--- a/reserve/renewscript.pl
+++ b/reserve/renewscript.pl
@@ -1,6 +1,5 @@
 #!/usr/bin/perl
 
-# $Id$
 
 #written 18/1/2000 by chris@katipo.co.nz
 #script to renew items from the web
@@ -19,48 +18,95 @@
 # 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, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA  02111-1307 USA
-
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+use strict;
+use warnings;
 use CGI;
-# use C4::Circulation::Renewals2;
-use C4::Circulation::Circ2;
-#get input
-my $input= new CGI;
-#print $input->header;
+use C4::Circulation;
+use C4::Auth;
+use URI::Escape;
+use Koha::DateUtils;
+my $input = new CGI;
 
-#print $input->dump;
+#Set Up User_env
+# And assures user is loggedin  and has correct accreditations.
 
-my @names=$input->param();
-my $count=@names;
-my %data;
+my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+    {
+        template_name   => "members/moremember.tmpl",
+        query           => $input,
+        type            => "intranet",
+        authnotrequired => 0,
+        flagsrequired   => { circulate => 'circulate_remaining_permissions' },
+        debug           => 0,
+    }
+);
 
-for (my $i=0;$i<$count;$i++){
-  if ($names[$i] =~ /renew/){
-    my $temp=$names[$i];
-    $temp=~ s/renew_item_//;
-    $data{$temp}=$input->param($names[$i]);
-  }
+#
+# find items to renew, all items or a selection of items
+#
+
+my @data;
+if ($input->param('renew_all')) {
+    @data = $input->param('all_items[]');
 }
-my %env;
-my $destination = $input->param("destination");
-my $cardnumber = $input->param("cardnumber");
-my $bornum=$input->param("bornum");
-while ( my ($itemno, $value) = each %data) {
-#    warn "$itemno = $value\n";
-   if ($value eq 'y'){
-     #means we want to renew this item
-     #check its status
-     my $status=renewstatus(\%env,$bornum,$itemno);
-     if ($status == 1){
-       renewbook(\%env,$bornum,$itemno);
-     }
-   }
+else {
+    @data = $input->param('items[]');
 }
 
-if($destination eq "circ"){
-       print $input->redirect("/cgi-bin/koha/circ/circulation.pl?findborrower=$cardnumber");
+my @barcodes;
+if ($input->param('return_all')) {
+    @barcodes = $input->param('all_barcodes[]');
 } else {
-       print $input->redirect("/cgi-bin/koha/members/moremember.pl?bornum=$bornum");
-}
\ No newline at end of file
+    @barcodes = $input->param('barcodes[]');
+}
+
+my $branch=$input->param('branch');
+my $datedue;
+if ($input->param('newduedate')){
+    $datedue = dt_from_string($input->param('newduedate');
+}
+
+# warn "barcodes : @barcodes";
+#
+# renew items
+#
+my $cardnumber = $input->param("cardnumber");
+my $borrowernumber = $input->param("borrowernumber");
+my $exemptfine = $input->param("exemptfine") || 0;
+my $override_limit = $input->param("override_limit") || 0;
+my $failedrenews = q{};
+foreach my $itemno (@data) {
+    # check status before renewing issue
+       my ($renewokay,$error) = CanBookBeRenewed($borrowernumber,$itemno,$override_limit);
+    if ($renewokay){
+        AddRenewal($borrowernumber,$itemno,$branch,$datedue);
+    }
+       else {
+               $failedrenews.="&failedrenew=$itemno";        
+       }
+}
+my $failedreturn = q{};
+foreach my $barcode (@barcodes) {
+    # check status before renewing issue
+   my ( $returned, $messages, $issueinformation, $borrower ) = 
+    AddReturn($barcode, $branch, $exemptfine);
+   $failedreturn.="&failedreturn=$barcode" unless ($returned);
+}
+
+#
+# redirection to the referrer page
+#
+if ($input->param('destination') eq "circ"){
+    $cardnumber = uri_escape($cardnumber);
+    print $input->redirect(
+        '/cgi-bin/koha/circ/circulation.pl?findborrower='.$cardnumber.$failedrenews.$failedreturn
+    );
+}
+else {
+    print $input->redirect(
+        '/cgi-bin/koha/members/moremember.pl?borrowernumber='.$borrowernumber.$failedrenews.$failedreturn
+    );
+}