bug 3093: Enhance placing of holds in the staff interface.

[koha.git] / reserve / placerequest.pl

diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl
index 96fdb7c..355b5b6 100755 (executable)
--- a/reserve/placerequest.pl
+++ b/reserve/placerequest.pl
@@ -34,6 +34,7 @@ use C4::Members;
 my $input = new CGI;
 #print $input->header;
 
+
 my @bibitems=$input->param('biblioitem');
 # FIXME I think reqbib does not exist anymore, it's used in line 82, to AddReserve of contraint type 'o'
 #       I bet it's a 2.x feature, reserving a given biblioitem, that is useless in Koha 3.0
@@ -49,9 +50,24 @@ my $type=$input->param('type');
 my $title=$input->param('title');
 my $borrowernumber=GetMember($borrower,'cardnumber');
 my $checkitem=$input->param('checkitem');
+
+my $multi_hold = $input->param('multi_hold');
+my $biblionumbers = $multi_hold ? $input->param('biblionumbers') : ($biblionumber . '/');
+my $bad_bibs = $input->param('bad_bibs');
+
+my %bibinfos = ();
+my @biblionumbers = split '/', $biblionumbers;
+foreach my $bibnum (@biblionumbers) {
+    my %bibinfo = ();
+    $bibinfo{title} = $input->param("title_$bibnum");
+    $bibinfo{rank} = $input->param("rank_$bibnum");
+    $bibinfos{$bibnum} = \%bibinfo;
+}
+
 my $found;
 
-#if we have an item selectionned, and the pickup branch is the same as the holdingbranch of the document, we force the value $rank and $found .
+# if we have an item selectionned, and the pickup branch is the same as the holdingbranch
+# of the document, we force the value $rank and $found .
 if ($checkitem ne ''){
     $rank[0] = '0' unless C4::Context->preference('ReservesNeedReturns');
     my $item = $checkitem;
@@ -62,31 +78,48 @@ if ($checkitem ne ''){
 }
 
 if ($type eq 'str8' && $borrowernumber ne ''){
-       my $count=@bibitems;
-       @bibitems=sort @bibitems;
-       my $i2=1;
-       my @realbi;
-       $realbi[0]=$bibitems[0];
-       for (my $i=1;$i<$count;$i++) {
-               my $i3=$i2-1;
-               if ($realbi[$i3] ne $bibitems[$i]) {
-                       $realbi[$i2]=$bibitems[$i];
-                       $i2++;
-               }
-       }
-       my $const;
-       if ($input->param('request') eq 'any'){
-            # place a request on 1st available
-            AddReserve($branch,$borrowernumber->{'borrowernumber'},$biblionumber,'a',\@realbi,$rank[0],$notes,$title,$checkitem,$found);
-       } elsif ($reqbib[0] ne ''){
-            # FIXME : elsif probably never reached, (see top of the script)
-            # place a request on a given item
-            AddReserve($branch,$borrowernumber->{'borrowernumber'},$biblionumber,'o',\@reqbib,$rank[0],$notes,$title,$checkitem, $found);
-       } else {
-            AddReserve($branch,$borrowernumber->{'borrowernumber'},$biblionumber,'a',\@realbi,$rank[0],$notes,$title,$checkitem, $found);
-       }
-       
-print $input->redirect("request.pl?biblionumber=$biblionumber");
+
+    foreach my $biblionumber (keys %bibinfos) {
+        my $count=@bibitems;
+        @bibitems=sort @bibitems;
+        my $i2=1;
+        my @realbi;
+        $realbi[0]=$bibitems[0];
+        for (my $i=1;$i<$count;$i++) {
+            my $i3=$i2-1;
+            if ($realbi[$i3] ne $bibitems[$i]) {
+                $realbi[$i2]=$bibitems[$i];
+                $i2++;
+            }
+        }
+        my $const;
+
+        if ($multi_hold) {
+            my $bibinfo = $bibinfos{$biblionumber};
+            AddReserve($branch,$borrowernumber->{'borrowernumber'},$biblionumber,'a',[$biblionumber],
+                       $bibinfo->{rank},$notes,$bibinfo->{title},$checkitem,$found);
+        } else {
+            if ($input->param('request') eq 'any'){
+                # place a request on 1st available
+                AddReserve($branch,$borrowernumber->{'borrowernumber'},$biblionumber,'a',\@realbi,$rank[0],$notes,$title,$checkitem,$found);
+            } elsif ($reqbib[0] ne ''){
+                # FIXME : elsif probably never reached, (see top of the script)
+                # place a request on a given item
+                AddReserve($branch,$borrowernumber->{'borrowernumber'},$biblionumber,'o',\@reqbib,$rank[0],$notes,$title,$checkitem, $found);
+            } else {
+                AddReserve($branch,$borrowernumber->{'borrowernumber'},$biblionumber,'a',\@realbi,$rank[0],$notes,$title,$checkitem, $found);
+            }
+        }
+    }
+
+    if ($multi_hold) {
+        if ($bad_bibs) {
+            $biblionumbers .= $bad_bibs;
+        }
+        print $input->redirect("request.pl?biblionumbers=$biblionumbers&multi_hold=1");
+    } else {
+        print $input->redirect("request.pl?biblionumber=$biblionumber");
+    }
 } elsif ($borrowernumber eq ''){
        print $input->header();
        print "Invalid card number please try again";