From f60d3837ee036e53df9ae1486d39f4a45a09e460 Mon Sep 17 00:00:00 2001
From: Alex Arnaud <alex.arnaud@biblibre.com>
Date: Mon, 1 Feb 2016 18:20:50 +0100
Subject: [PATCH] Bz 15565 - Place multiple holds at once for the same record
 on staff and OPAC.

From staff interface, place several item-level holds for the same user
on the same record at once:
    - check that you are warned if you exceed "Holds per record (count)"
      (circulation rules) for this record,
  - check also maxReserves system preference.

From opac, place several item-level and record level holds for the same
user on the same record at once:
  - check that can't exceed "Holds per record (count)" for this record,
  - check also maxReserves system preference.

Do the same in OPAC and staff by selecting several record (multi-hold).
---
 .../prog/en/modules/reserve/request.tt             |  34 ++--
 .../opac-tmpl/bootstrap/en/modules/opac-reserve.tt | 160 +++++++++---------
 koha-tmpl/opac-tmpl/bootstrap/less/opac.less       |   3 +
 opac/opac-reserve.pl                               | 187 ++++++++++-----------
 reserve/placerequest.pl                            |  32 ++--
 5 files changed, 196 insertions(+), 220 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
index 2dbd482..913e83f 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -22,6 +22,8 @@ var override_items = {[% FOREACH bibitemloo IN bibitemloop %][% FOREACH itemloo
 [% END %][% END %][% END %]
 };
 var MSG_NO_ITEMS_AVAILABLE = _("A hold cannot be requested on any of these items.");
+var MSG_EXCEEDED_HOLDS_PER_RECORD = _("Patron has exceeded the number of holds for this record.");
+var remainingHolds = "[% remaining_holds_for_record %]";
 
 $(document).ready(function() {
     [% IF AutoResumeSuspendedHolds %]
@@ -148,19 +150,23 @@ function checkMultiHold() {
 		});
 		}
 	});
- 	$("input[name=checkitem]").click(function() {
-		onechecked = 0;
-		$("input[name=checkitem]").each(function() {
-			if(this.checked){
-				onechecked = 1;
-			}
-		});
-		if(onechecked == 1){
+    $("input[name=checkitem]").click(function() {
+        numchecked = 0;
+        $("input[name=checkitem]").each(function() {
+            if(this.checked){
+                numchecked++;
+            }
+        });
+        if(numchecked > 0){
             $("#requestany").removeAttr("checked");
-		} else {
-			$("#requestany").attr("checked","checked");
-		}
-	});
+        } else {
+            $("#requestany").attr("checked","checked");
+        }
+
+        if (remainingHolds - numchecked < 0) {
+            alert(MSG_EXCEEDED_HOLDS_PER_RECORD);
+        }
+    });
 
     $(".clear-date").on("click",function(e){
         e.preventDefault();
@@ -512,9 +518,9 @@ function checkMultiHold() {
                         Hold must be record level
                     </span>
                 [% ELSIF ( itemloo.available ) %]
-                    <input type="radio" name="checkitem" value="[% itemloo.itemnumber %]" />
+                    <input type="checkbox" name="checkitem" value="[% itemloo.itemnumber %]" />
                 [% ELSIF ( itemloo.override ) %]
-                    <input type="radio" name="checkitem" class="needsoverride" value="[% itemloo.itemnumber %]" />
+                    <input type="checkbox" name="checkitem" class="needsoverride" value="[% itemloo.itemnumber %]" />
                     <i class="fa fa-exclamation-triangle fa-lg" style="color:gold" alt="Requires override of hold policy"/></i>
                 [% ELSE %]
                     <span class="error">
diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt
index 516ada5..5246321 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt
@@ -124,20 +124,18 @@
                         <form action="/cgi-bin/koha/opac-reserve.pl" method="post" id="hold-request-form">
                             <input type="hidden" name="place_reserve" value="1"/>
                             <!-- These values are set dynamically by js -->
-                            <input type="hidden" name="biblionumbers" id="biblionumbers"/>
-                            <input type="hidden" name="selecteditems" id="selections"/>
                             <div id="bigloop">
 
                                 [% FOREACH bibitemloo IN bibitemloop %]
                                     <div class="holdrow">
+                                        <input id="remaining_holds_[% bibitemloo.biblionumber %]" type="hidden" value="[% bibitemloo.remaining_holds_for_record %]"/>
                                         <p>
                                             [% IF ( bibitemloo.holdable ) %]
-                                                <input class="reserve_mode" name="reserve_mode" type="hidden" value="single"/>
-                                                <input class="single_bib" name="single_bib" type="hidden" value="[% bibitemloo.biblionumber %]"/>
-                                                <span class="confirmjs_hold" title="[% bibitemloo.biblionumber %]" style="padding:.3em"></span>
-                                                <span class="confirm_nonjs">
-                                                    <input type="radio" class="confirmbox checkitem [% bibitemloo.biblionumber %]" name="[% bibitemloo.biblionumber %]" checked="checked" id="single_[% bibitemloo.biblionumber %]" value="any" />
-                                                    <label class="confirm_label" for="single_[% bibitemloo.biblionumber %]">Place a hold on </label>
+                                                <span title="[% bibitemloo.biblionumber %]" style="padding:.3em">
+                                                    <label>
+                                                        <input class="biblionumbers" name="biblionumbers" type="checkbox" value="[% bibitemloo.biblionumber %]" checked="checked">
+                                                        Place a hold on
+                                                    </label>
                                                 </span>
                                             [% END # / bibitemloo.holdable %]
 
@@ -204,7 +202,7 @@
                                                             <li class="branch">
                                                                 <label for="branch_[% bibitemloo.biblionumber %]">Pick up location:</label>
                                                                 [% UNLESS ( bibitemloo.holdable ) %]
-                                                                    <select name="branch" id="branch_[% bibitemloo.biblionumber %]" disabled="disabled">
+                                                                    <select name="branch_[% bibitemloo.biblionumber %]" id="branch_[% bibitemloo.biblionumber %]" disabled="disabled">
                                                                         [% FOREACH branchloo IN bibitemloo.branchloop %]
                                                                             [% IF ( branchloo.selected ) %]
                                                                                 <option value="[% branchloo.branchcode %]" selected="selected">[% branchloo.branchname %]</option>
@@ -214,7 +212,7 @@
                                                                         [% END %]
                                                                     </select>
                                                                 [% ELSE %]
-                                                                    <select name="branch" id="branch_[% bibitemloo.biblionumber %]">
+                                                                    <select name="branch_[% bibitemloo.biblionumber %]" id="branch_[% bibitemloo.biblionumber %]">
                                                                         [% FOREACH branchloo IN bibitemloo.branchloop %]
                                                                             [% IF ( branchloo.selected ) %]
                                                                                 <option value="[% branchloo.branchcode %]" selected="selected">[% branchloo.branchname %]</option>
@@ -261,10 +259,8 @@
 
                                                         <li>
                                                         </li>
-                                                        [% IF bibitemloo.itemholdable %]
                                                             <!-- ITEM HOLDS -->
                                                             <li class="lradio place_on_type" style="display:none;">
-                                                                [% IF NOT bibitemloo.force_hold %]
                                                                     <label class="radio inline" for="reqany_[% bibitemloo.biblionumber %]">Next available item</label>
                                                                     <input type="radio" name="reqtype_[% bibitemloo.biblionumber %]"
                                                                             id="reqany_[% bibitemloo.biblionumber %]"
@@ -272,7 +268,6 @@
                                                                             value="Any"
                                                                             checked="checked"
                                                                     />
-                                                                [% END %]
                                                                 <label class="radio inline" for="reqspecific_[% bibitemloo.biblionumber %]">A specific item</label>
                                                                 <input type="radio" name="reqtype_[% bibitemloo.biblionumber %]"
                                                                        id="reqspecific_[% bibitemloo.biblionumber %]"
@@ -280,10 +275,23 @@
                                                                        value="Specific"
                                                                 />
                                                             </li>
-                                                        [% END # / IF bibitemloo.itemholdable %]
                                                     </ul>
 
-                                                    [% IF bibitemloo.itemholdable %]
+                                                    [% IF bibitemloo.remaining_holds_for_record > 1 %]
+                                                        [% SET count = 1 %]
+                                                        <div id="holds_to_place_[% bibitemloo.biblionumber %]" class="hold-options holds-to-place">
+                                                            <label>Holds to place (count)</label>
+                                                            <select name="holds_to_place_count_[% bibitemloo.biblionumber %]">
+                                                            [% WHILE count <= bibitemloo.remaining_holds_for_record %]
+                                                                <option value="[% count %]">[% count %]</option>
+                                                                [% SET count = count + 1 %]
+                                                            [% END %]
+                                                            </select>
+                                                        </div>
+                                                    [% ELSE %]
+                                                        <input type="hidden" name="holds_to_place_count_[% bibitemloo.biblionumber %]" value="1" />
+                                                    [% END %]
+
                                                         <table class="copiesrow table table-bordered table-striped" id="copiesrow_[% bibitemloo.biblionumber %]">
                                                             <caption>Select a specific item:</caption>
                                                             <tr>
@@ -307,9 +315,9 @@
                                                                 <tr class="[% itemLoo.backgroundcolor %]">
                                                                     <td class="copynumber">
                                                                         [% IF ( itemLoo.available ) %]
-                                                                            <input type="radio" class="checkitem checkitem_[% bibitemloo.biblionumber %]" name="checkitem_[% bibitemloo.biblionumber %]" value="[% itemLoo.itemnumber %]" />
+                                                                            <input type="checkbox" class="checkitem checkitem_[% bibitemloo.biblionumber %]" name="checkitem_[% bibitemloo.biblionumber %]" value="[% itemLoo.itemnumber %]" />
                                                                         [% ELSE %]
-                                                                            <input disabled="disabled" type="radio" class="checkitem" name="checkitem" value="[% itemLoo.itemnumber %]"
+                                                                            <input disabled="disabled" type="checkbox" class="checkitem" name="checkitem" value="[% itemLoo.itemnumber %]"
                                                                                    style="display:none;" />
                                                                             <img src="[% interface %]/lib/famfamfam/silk/cross.png" alt="Cannot be put on hold" title="Cannot be put on hold" />
                                                                         [% END %]
@@ -383,7 +391,6 @@
                                                                 </tr>
                                                             [% END # / FOREACH itemLoo IN bibitemloo.itemLoop%]
                                                         </table> <!-- / #copiesrow_[% bibitemloo.biblionumber %] -->
-                                                    [% END # / IF ( bibitemloo.itemholdable )%]
                                                 </div> <!-- / #hold-options-[% bibitemloo.biblionumber %] -->
                                             </fieldset>
                                         [% END # / IF ( bibitemloo.holdable ) %]
@@ -406,6 +413,7 @@
 <script type="text/javascript">
 // <![CDATA[
     var MSG_NO_ITEM_SELECTED = _("Expecting a specific item selection.");
+    var MSG_MAX_HOLDS_EXCEEDED = _("Maximum number of reserve exceded.");
     var ForceHoldNotesReasons=new Array(
        _("This title consists of multiple volumes/parts. Please indicate which part you need. Clicking on specific item information may be helpful."),
        "*** Add a new reason above this line ***" );
@@ -419,8 +427,8 @@
 
     // Select the first item available
     function select_first_available(id){
-        var radios = $("input:radio[name='checkitem_" + id + "']");
-        $(radios).first().attr("checked", "checked");
+        var checkboxes = $("input:checkbox[name='checkitem_" + id + "']");
+        $(checkboxes).first().attr("checked", "checked");
     }
 
     $(document).ready(function() {
@@ -441,14 +449,16 @@
             }
         });
 
-        // click on a first td check the confirmjs checkbox
-        $("td.hold").click(function(e){
-          if(e.target.tagName.toLowerCase() == 'td'){
-            $(this).find("input.confirmjs").each( function() {
-               $(this).attr('checked', !$(this).attr('checked'));
-               $(this).change();
-            });
-          }
+        $('.checkitem').click(function(e) {
+            var bibNum = suffixOf($(this).attr("name"), "_");
+            var nbCheked = $("input[name='checkitem_"+bibNum+"']:checked").length;
+            var remaining = $("input[id='remaining_holds_"+bibNum+"']").val();
+            if (nbCheked >= remaining) {
+                $("input[name='checkitem_"+bibNum+"']:not(:checked)").attr('disabled', 'disabled');
+            }
+            else {
+                $('.checkitem_'+bibNum).attr('disabled', false);
+            }
         });
 
         $(".toggle-hold-options").on("click",function(e){
@@ -463,12 +473,21 @@
         // Hides all 'specific copy' table rows on load.
         $(".copiesrow").hide();
 
+        $(".holds-to-place").show();
         [% FOREACH bibitemloo IN bibitemloop %]
-          [% IF bibitemloo.force_hold %]
-            $("#toggle-hold-options-[% bibitemloo.biblionumber %]").click();
-            $("#reqspecific_[% bibitemloo.biblionumber %]").click();
-            $("#copiesrow_[% bibitemloo.biblionumber %]").show();
-          [% END %]
+            [% IF bibitemloo.force_hold %]
+                [% IF bibitemloo.force_hold == 'item' %]
+                    $("#toggle-hold-options-[% bibitemloo.biblionumber %]").click();
+                    $("#reqspecific_[% bibitemloo.biblionumber %]").attr('checked', true);
+                    $("#holds_to_place_[% bibitemloo.biblionumber %]").hide();
+                    $("#copiesrow_[% bibitemloo.biblionumber %]").show();
+                [% ELSIF bibitemloo.force_hold == 'record' %]
+                    $("#toggle-hold-options-[% bibitemloo.biblionumber %]").click();
+                    $("#reqany_[% bibitemloo.biblionumber %]").attr('checked', true);
+                [% END %]
+                $("#reqany_[% bibitemloo.biblionumber %]").attr('disabled', 'disabled');
+                $("#reqspecific_[% bibitemloo.biblionumber %]").attr('disabled', 'disabled');
+            [% END %]
         [% END %]
 
         // Insert reasons for forced hold notes
@@ -494,40 +513,37 @@
             $("#" + op + fieldID).val("");
         });
 
-        // Replace non-JS single-selection with multi-selection capability.
-        $(".reserve_mode").val("multi");
-        $(".confirm_nonjs").remove();
-        $(".confirmjs_hold").each(function(){
-            var bib = $(this).attr("title");
-            var html = "<label><input type =\"checkbox\" class=\"confirmjs\" checked=\"checked\"";
-            html += "value=\"" + bib + "\" id=\"" + bib + "\" /> " + _("Place a hold on") + " </label> ";
-            $(this).html(html);
-        });
-        $(".confirmjs_nohold").each(function(){
-            var bib = $(this).attr("title");
-            var html = "<label><input type =\"checkbox\" class=\"confirmjs\" disabled=\"disabled\"";
-            html += "value=\"" + bib + "\" id=\"" + bib + "\" />" + _("Place a hold on: ") + "</label>";
-            $(this).html(html);
-        });
-
         // expand or collapse the copiesrow tr
         function toggle_copiesrow(biblioNum) {
             var checkbox = $("input:checkbox[value='"+biblioNum+"']");
             newCopiesRowId = "#copiesrow_" + biblioNum;
+            holdsToPlaceCountId = "#holds_to_place_" + biblioNum;
             var select_specific = $("#reqspecific_"+biblioNum).is(":checked");
             // If the checkbox is checked AND we want a specific item, we display the items block
             if ( $(checkbox).is(":checked") && select_specific ) {
                 $(newCopiesRowId).show();
+                $(holdsToPlaceCountId).hide();
             } else {
                 $(newCopiesRowId).hide();
+                $(holdsToPlaceCountId).show();
              }
         };
 
-        $("#place_on_hdr").show();
+        function total_requested() {
+            var total = 0;
+            $("input[name='biblionumbers']:checked").each(function() {
+                var biblioNum = $(this).val();
+                if ($("#reqspecific_" + biblioNum + ":checked").size() > 0) {
+                    total += parseInt($("input[name='checkitem_'"+biblioNum+"]:checked").length);
+                } else {
+                    total += parseInt($("select[name='holds_to_place_count_"+biblioNum+"']").val());
+                }
+            });
+            return total;
+        }
 
         $(".place_on_type").show();
         // onload, selectany is checked
-        $(".selectany").attr("checked", "checked");
 
         // If the user is *allowed* to choose a specific item
         // The first one is preselected
@@ -536,8 +552,8 @@
             select_first_available(id);
         });
 
-        // On confirmsjs change
-        $(".confirmjs").change(function(){
+        // On biblionumbers change
+        $(".biblionumbers").change(function(){
             var id = suffixOf($(this).attr("id"), "_");
             // If I m checked, I enable radio buttons
             if ( $(this).is(":checked") ) {
@@ -559,35 +575,22 @@
             toggle_copiesrow(id);
         });
 
-        // Show or hide holds notes
-        $(".shownotes").click(function(){
-            biblioNum = suffixOf($(this).attr("id"), "_");
-            $("#notesrow_"+biblioNum).toggle();
-        });
-
         // When 'Place Hold' button is clicked
         $(".placehold").click(function(){
-            var biblionumbers = "";
-            var selections = "";
-
-            [% IF new_reserves_allowed %]
-                if ($(".confirmjs:checked").size() > [% new_reserves_allowed %] ) {
-                    alert(MSG_MAX_HOLDS_EXCEEDED);
-                    return false;
-                }
-            [% END %]
+            if (total_requested() + [% reserves_count %] > [% maxreserves %]) {
+                alert(MSG_MAX_HOLDS_EXCEEDED);
+                return false;
+            }
 
-            if ($(".confirmjs:checked").size() == 0) {
+            if ($("input[name='biblionumbers']:checked").size() == 0) {
                 alert(MSG_NO_RECORD_SELECTED);
                 return false;
             }
 
             // Find the items with the 'Hold' box checked
             var badBib = null;
-            $(".confirmjs:checked").each(function() {
+            $("input[name='biblionumbers']:checked").each(function() {
                 var biblioNum = $(this).val();
-                biblionumbers += biblioNum + "/";
-                selections += biblioNum + "/";
 
                 // If the 'specific copy' radio button is checked
                 if ($("#reqspecific_" + biblioNum + ":checked").size() > 0) {
@@ -596,18 +599,8 @@
                     if ($(item).size() == 0) {
                         badBib = biblioNum;
                         return false;
-                    } else {
-                      selections += $(item).val();
                     }
                 }
-                selections += "/";
-
-                // Add the pickup location
-                var branchSel = $("#branch_" + biblioNum);
-                if (branchSel.size() > 0) {
-                    selections += $(branchSel).val();
-                }
-                selections += "/";
                 return true;
             });
 
@@ -616,9 +609,6 @@
                 return false;
             }
 
-            $("#selections").val(selections);
-            $("#biblionumbers").val(biblionumbers);
-
             return true;
         });
 
diff --git a/koha-tmpl/opac-tmpl/bootstrap/less/opac.less b/koha-tmpl/opac-tmpl/bootstrap/less/opac.less
index 447e255..148535c 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/less/opac.less
+++ b/koha-tmpl/opac-tmpl/bootstrap/less/opac.less
@@ -2375,6 +2375,9 @@ td img {
     margin: 1em 0;
     padding: .5em;
 }
+.hold-option select {
+    width: auto;
+}
 .copiesrow {
     clear : both;
 }
diff --git a/opac/opac-reserve.pl b/opac/opac-reserve.pl
index 4e1e93f..59f9048 100755
--- a/opac/opac-reserve.pl
+++ b/opac/opac-reserve.pl
@@ -91,35 +91,19 @@ if ($borr->{reservefee} > 0){
 my $branches = GetBranches();
 my $itemTypes = GetItemTypes();
 
-# There are two ways of calling this script, with a single biblio num
-# or multiple biblio nums.
-my $biblionumbers = $query->param('biblionumbers');
-my $reserveMode = $query->param('reserve_mode');
-if ($reserveMode && ($reserveMode eq 'single')) {
-    my $bib = $query->param('single_bib');
-    $biblionumbers = "$bib/";
-}
-if (! $biblionumbers) {
-    $biblionumbers = $query->param('biblionumber');
-}
-
-if ((! $biblionumbers) && (! $query->param('place_reserve'))) {
-    $template->param(message=>1, no_biblionumber=>1);
-    &get_out($query, $cookie, $template->output);
+# Coming from opac-search biblionumbers param is a string, coming from
+# opac-reserve, it's an array.
+my @biblionumbers = ref($query->param('biblionumbers')) eq 'Array'
+    ? $query->param('biblionumbers')
+    : split(/\//, $query->param('biblionumbers'));
+
+unless (@biblionumbers) {
+    push(@biblionumbers, $query->param('biblionumber'));
 }
 
 # Pass the numbers to the page so they can be fed back
 # when the hold is confirmed. TODO: Not necessary?
-$template->param( biblionumbers => $biblionumbers );
-
-# Each biblio number is suffixed with '/', e.g. "1/2/3/"
-my @biblionumbers = split /\//, $biblionumbers;
-if (($#biblionumbers < 0) && (! $query->param('place_reserve'))) {
-    # TODO: New message?
-    $template->param(message=>1, no_biblionumber=>1);
-    &get_out($query, $cookie, $template->output);
-}
-
+$template->param( biblionumbers => @biblionumbers );
 
 # pass the pickup branch along....
 my $branch = $query->param('branch') || $borr->{'branchcode'} || C4::Context->userenv->{branch} || '' ;
@@ -194,39 +178,22 @@ if ( $query->param('place_reserve') ) {
         $reserve_cnt = GetReservesFromBorrowernumber( $borrowernumber );
     }
 
-    # List is composed of alternating biblio/item/branch
-    my $selectedItems = $query->param('selecteditems');
-
-    if ($query->param('reserve_mode') eq 'single') {
-        # This indicates non-JavaScript mode, so there was
-        # only a single biblio number selected.
-        my $bib = $query->param('single_bib');
-        my $item = $query->param("checkitem_$bib");
-        if ($item eq 'any') {
-            $item = '';
-        }
-        my $branch = $query->param('branch');
-        $selectedItems = "$bib/$item/$branch/";
-    }
-
-    $selectedItems =~ s!/$!!;
-    my @selectedItems = split /\//, $selectedItems, -1;
-
-    # Make sure there is a biblionum/itemnum/branch triplet for each item.
-    # The itemnum can be 'any', meaning next available.
-    my $selectionCount = @selectedItems;
-    if (($selectionCount == 0) || (($selectionCount % 3) != 0)) {
-        $template->param(message=>1, bad_data=>1);
-        &get_out($query, $cookie, $template->output);
-    }
-
     my $failed_holds = 0;
-    while (@selectedItems) {
-        my $biblioNum = shift(@selectedItems);
-        my $itemNum   = shift(@selectedItems);
-        my $branch    = shift(@selectedItems);    # i.e., branch code, not name
-
-        my $canreserve = 0;
+    foreach my $biblioNum (@biblionumbers) {
+        my $itemNum = $query->param("checkitem_$biblioNum");
+        my @itemnumbers = $query->param("checkitem_$biblioNum");
+        my $branch = $query->param("branch_$biblioNum");
+        my $reqtype = $query->param("reqtype_$biblioNum");
+        my $holds_to_place_count = $query->param("holds_to_place_count_$biblioNum") || 1;
+        my $notes = $query->param('notes_'.$biblioNum) || '';
+        my $nbRequested = $reqtype eq 'Specific' ? @itemnumbers : $holds_to_place_count;
+
+        my $canreserve = 1;
+        if (   $maxreserves
+            && $reserve_cnt + $nbRequested >= $maxreserves )
+        {
+            $canreserve = 0;
+        }
 
         my $singleBranchMode = Koha::Libraries->search->count == 1;
         if ( $singleBranchMode || !$OPACChooseBranch )
@@ -234,14 +201,6 @@ if ( $query->param('place_reserve') ) {
             $branch = $borr->{'branchcode'};
         }
 
-#item may belong to a host biblio, if yes change biblioNum to hosts bilbionumber
-        if ( $itemNum ne '' ) {
-            my $hostbiblioNum = GetBiblionumberFromItemnumber($itemNum);
-            if ( $hostbiblioNum ne $biblioNum ) {
-                $biblioNum = $hostbiblioNum;
-            }
-        }
-
         my $biblioData = $biblioDataHash{$biblioNum};
         my $found;
 
@@ -255,44 +214,60 @@ if ( $query->param('place_reserve') ) {
 
         my $expiration_date = $query->param("expiration_date_$biblioNum");
 
-      # If a specific item was selected and the pickup branch is the same as the
-      # holdingbranch, force the value $rank and $found.
+        # If a specific item was selected and the pickup branch is the same as the
+        # holdingbranch, force the value $rank and $found.
         my $rank = $biblioData->{rank};
-        if ( $itemNum ne '' ) {
-            $canreserve = 1 if CanItemBeReserved( $borrowernumber, $itemNum ) eq 'OK';
-            $rank = '0' unless C4::Context->preference('ReservesNeedReturns');
-            my $item = GetItem($itemNum);
-            if ( $item->{'holdingbranch'} eq $branch ) {
-                $found = 'W'
-                  unless C4::Context->preference('ReservesNeedReturns');
+        if ( $reqtype eq 'Specific') {
+            foreach my $itemnumber (@itemnumbers) {
+                my $biblionumber = $biblioNum;
+
+                #item may belong to a host biblio, if yes change biblioNum to hosts bilbionumber
+                my $hostbiblioNum = GetBiblionumberFromItemnumber($itemnumber);
+                if ( $hostbiblioNum ne $biblioNum ) {
+                    $biblionumber = $hostbiblioNum;
+                }
+
+                $canreserve = 0 unless CanItemBeReserved( $borrowernumber, $itemnumber ) eq 'OK';
+                $rank = '0' unless C4::Context->preference('ReservesNeedReturns');
+                my $item = GetItem($itemnumber);
+                if ( $item->{'holdingbranch'} eq $branch ) {
+                    $found = 'W'
+                      unless C4::Context->preference('ReservesNeedReturns');
+                }
+
+                if ($canreserve) {
+                    my $reserve_id = AddReserve(
+                        $branch,         $borrowernumber,
+                        $biblionumber,
+                        [$biblionumber], $rank,
+                        $startdate,      $expiration_date,
+                        $notes,          $biblioData->{title},
+                        $itemnumber,     $found
+                    );
+                    $failed_holds++ unless $reserve_id;
+                    ++$reserve_cnt;
+                }
             }
+            print $query->redirect("/cgi-bin/koha/opac-user.pl?" . ( $failed_holds ? "failed_holds=$failed_holds" : q|| ) . "#opac-user-holds");
+            exit;
         }
-        else {
-            $canreserve = 1 if CanBookBeReserved( $borrowernumber, $biblioNum ) eq 'OK';
-
-            # Inserts a null into the 'itemnumber' field of 'reserves' table.
-            $itemNum = undef;
-        }
-        my $notes = $query->param('notes_'.$biblioNum)||'';
 
-        if (   $maxreserves
-            && $reserve_cnt >= $maxreserves )
-        {
-            $canreserve = 0;
-        }
+        $canreserve = 0 unless CanBookBeReserved( $borrowernumber, $biblioNum ) eq 'OK';
 
         # Here we actually do the reserveration. Stage 3.
         if ($canreserve) {
-            my $reserve_id = AddReserve(
-                $branch,      $borrowernumber,
-                $biblioNum,
-                [$biblioNum], $rank,
-                $startdate,   $expiration_date,
-                $notes,       $biblioData->{title},
-                $itemNum,     $found
-            );
-            $failed_holds++ unless $reserve_id;
-            ++$reserve_cnt;
+            for ( my $i = 0 ; $i < $holds_to_place_count ; $i++ ) {
+                my $reserve_id = AddReserve(
+                    $branch,      $borrowernumber,
+                    $biblioNum,
+                    [$biblioNum], $rank,
+                    $startdate,   $expiration_date,
+                    $notes,       $biblioData->{title},
+                    $itemNum,     $found
+                );
+                $failed_holds++ unless $reserve_id;
+                ++$reserve_cnt;
+            }
         }
     }
 
@@ -351,6 +326,10 @@ if ( $maxreserves && ( $reserves_count >= $maxreserves ) ) {
 }
 
 unless ( $noreserves ) {
+    $template->param(
+        maxreserves => $maxreserves,
+        reserves_count => $reserves_count
+    );
     my $requested_reserves_count = scalar( @biblionumbers );
     if ( $maxreserves && ( $reserves_count + $requested_reserves_count > $maxreserves ) ) {
         $template->param( new_reserves_allowed => $maxreserves - $reserves_count );
@@ -519,7 +498,7 @@ foreach my $biblioNum (@biblionumbers) {
             if ( my $hold_allowed = OPACItemHoldsAllowed( $itemInfo, $borr ) ) {
                 $itemLoopIter->{available} = 1;
                 $numCopiesOPACAvailable++;
-                $biblioLoopIter{force_hold} = 1 if $hold_allowed eq 'F';
+                $biblioLoopIter{force_hold} = 'item' if $hold_allowed eq 'F';
             }
             $numCopiesAvailable++;
         }
@@ -557,18 +536,24 @@ foreach my $biblioNum (@biblionumbers) {
     # patron placed a record level hold, all the holds the patron places must
     # be record level. If the patron placed an item level hold, all holds
     # the patron places must be item level
-    my $forced_hold_level = Koha::Holds->search(
+    my $holds = Koha::Holds->search(
         {
             borrowernumber => $borrowernumber,
             biblionumber   => $biblioNum,
             found          => undef,
         }
-    )->forced_hold_level();
+    );
+    my $forced_hold_level = $holds->forced_hold_level();
     if ($forced_hold_level) {
-        $biblioLoopIter{force_hold}   = 1 if $forced_hold_level eq 'item';
-        $biblioLoopIter{itemholdable} = 0 if $forced_hold_level eq 'record';
+        #$biblioLoopIter{force_hold}   = 1 if $forced_hold_level eq 'item';
+        #$biblioLoopIter{itemholdable} = 0 if $forced_hold_level eq 'record';
+        $biblioLoopIter{force_hold} = $forced_hold_level;
     }
 
+    my $max_holds_for_record = GetMaxPatronHoldsForRecord( $borrowernumber, $biblioNum );
+    my $remaining_holds_for_record = $max_holds_for_record - $holds->count();
+    $biblioLoopIter{remaining_holds_for_record} = $remaining_holds_for_record;
+
 
     push @$biblioLoop, \%biblioLoopIter;
 
diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl
index b7335ae..e1aa890 100755
--- a/reserve/placerequest.pl
+++ b/reserve/placerequest.pl
@@ -48,13 +48,14 @@ my @rank=$input->param('rank-request');
 my $type=$input->param('type');
 my $title=$input->param('title');
 my $borrower=GetMember('borrowernumber'=>$borrowernumber);
-my $checkitem=$input->param('checkitem');
+my @checkitems=$input->param('checkitem');
 my $expirationdate = $input->param('expiration_date');
 
 my $multi_hold = $input->param('multi_hold');
 my $biblionumbers = $multi_hold ? $input->param('biblionumbers') : ($biblionumber . '/');
 my $bad_bibs = $input->param('bad_bibs');
 my $holds_to_place_count = $input->param('holds_to_place_count') || 1;
+$holds_to_place_count = scalar(@checkitems) if scalar(@checkitems);
 
 my %bibinfos = ();
 my @biblionumbers = split '/', $biblionumbers;
@@ -65,17 +66,15 @@ foreach my $bibnum (@biblionumbers) {
     $bibinfos{$bibnum} = \%bibinfo;
 }
 
-my $found;
+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 (defined $checkitem && $checkitem ne ''){
-    $holds_to_place_count = 1;
+foreach my $checkitem (@checkitems) {
+    # 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 .
     $rank[0] = '0' unless C4::Context->preference('ReservesNeedReturns');
-    my $item = $checkitem;
-    $item = GetItem($item);
+    my $item = GetItem($checkitem);
     if ( $item->{'holdingbranch'} eq $branch ){
-        $found = 'W' unless C4::Context->preference('ReservesNeedReturns');
+        push @found, 'W' unless C4::Context->preference('ReservesNeedReturns');
     }
 }
 
@@ -95,25 +94,18 @@ if ( $type eq 'str8' && $borrower ) {
             }
         }
 
-        if ( defined $checkitem && $checkitem ne '' ) {
-            my $item = GetItem($checkitem);
-            if ( $item->{'biblionumber'} ne $biblionumber ) {
-                $biblionumber = $item->{'biblionumber'};
-            }
-        }
-
         if ($multi_hold) {
             my $bibinfo = $bibinfos{$biblionumber};
             AddReserve( $branch, $borrower->{'borrowernumber'},
                 $biblionumber, [$biblionumber],
-                $bibinfo->{rank}, $startdate, $expirationdate, $notes, $bibinfo->{title}, $checkitem,
-                $found );
+                $bibinfo->{rank}, $startdate, $expirationdate, $notes, $bibinfo->{title});
         }
         else {
             for ( my $i = 0 ; $i < $holds_to_place_count ; $i++ ) {
+                my $item = GetItem($checkitems[$i]);
                 AddReserve( $branch, $borrower->{'borrowernumber'},
-                    $biblionumber, \@realbi, $rank[0], $startdate, $expirationdate, $notes, $title,
-                    $checkitem, $found );
+                    $item->{biblionumber}, \@realbi, $rank[0], $startdate, $expirationdate, $notes, $title,
+                    $checkitems[$i], $found[$i] );
             }
         }
     }
-- 
2.1.4