#1455 (placing reserve from OPAC)
authorPaul POULAIN <paul@koha-fr.org>
Wed, 3 Oct 2007 17:01:06 +0000 (19:01 +0200)
committerChris Cormack <crc@liblime.com>
Wed, 3 Oct 2007 19:58:55 +0000 (14:58 -0500)
SANOP hadn't ported reserve to OPAC yet. that's why opac reserves were no more working.

I've fixed that, but it need more testing...

Signed-off-by: Chris Cormack <crc@liblime.com>
koha-tmpl/opac-tmpl/prog/en/opac-reserve.tmpl
opac/opac-reserve.pl
reserve/placerequest.pl
reserve/request.pl

index 3a7cdf3..e4c1485 100644 (file)
 <!-- /TMPL_IF -->
 
 <!-- TMPL_IF NAME="select_item_types" -->
-<!-- The first time round you select which bibitems you want -->
-<form action="/cgi-bin/koha/opac-reserve.pl" method="post">
-<input type="hidden" name="item_types_selected" value="1">
-<input type="hidden" name="biblionumber" value="<!-- TMPL_VAR NAME="biblionumber" -->">
-<h2>Reservation Details</h2>
-<div class="form">
-<table>
-   <tr><th>Reserving: </th><td><span class="title"><!-- TMPL_VAR NAME="title" --></span><!-- TMPL_IF NAME="author" --> (<!-- TMPL_VAR NAME="author" -->)<!-- /TMPL_IF --></td></tr>
-    <tr><th>For:</th><td><!-- TMPL_LOOP NAME="USER_INFO" --><!-- TMPL_VAR NAME="firstname" --> <!-- TMPL_VAR NAME="surname" --> (<!-- TMPL_VAR NAME="cardnumber" -->)<!-- /TMPL_LOOP --></td></tr>
-
-    <tr><th>Pick up from:</th><td><select name="branch" id="branch">
-        <option>-- Please choose your branch --</option>
-        <!-- TMPL_LOOP NAME="CGIbranch" --><option value="<!-- TMPL_VAR NAME="value" -->"><!-- TMPL_VAR NAME="branch" --></option><!-- /TMPL_LOOP -->
-</select></td></tr>
-</table>
-
-<!-- TMPL_IF NAME="onlyone" -->
-<!-- If there is only one type -->
-
-<p><input type="submit" class="submit" value="Next &gt;&gt;">
-<input type="hidden" name="all" value="1"></p>
-</form>
-
-<!-- TMPL_ELSE -->
-<!-- If there is more than one type -->
-
-    <h4>Choose Either...</h4>
-        <p>
-            <input id="checknextavailable" type="checkbox" name="all" value="next-available">
-            <label for="checknextavailable">Next Available Copy (any format)</label>
+    <!-- The first time round you select which bibitems you want -->
+    <form action="/cgi-bin/koha/opac-reserve.pl" method="post">
+        <input type="hidden" name="item_types_selected" value="1">
+        <input type="hidden" name="biblionumber" value="<!-- TMPL_VAR NAME="biblionumber" -->">
+        <h2>Reservation Details</h2>
+        <div class="form">
+            <h3>Reserving: <span class="title"><!-- TMPL_VAR NAME="title" -->, <!-- TMPL_IF NAME="author" --> (<!-- TMPL_VAR NAME="author" -->)<!-- /TMPL_IF --></h3>
+            <p>For <!-- TMPL_LOOP NAME="USER_INFO" --><!-- TMPL_VAR NAME="firstname" --> <!-- TMPL_VAR NAME="surname" --> (<!-- TMPL_VAR NAME="cardnumber" -->)<!-- /TMPL_LOOP --></p>
+            
+        <!-- TMPL_LOOP name="bibitemloop" -->    
+            <p><span class="label">Item type:</span> <!-- TMPL_VAR NAME="description" --></p>
+            <!-- TMPL_IF NAME="publicationyear" --><p><span class="label">Publication year:</span> <!-- TMPL_VAR NAME="publicationyear" --></p><!-- /TMPL_IF -->
+            <table>
+            <tr>
+                <th>Barcode</th>
+                <th>Home Library</th>
+                <th>Holding Library</th>
+                <th>Call number</th>
+                <th>Information</th>
+                <th>Cancel a Reserve</th>
+            </tr>
+            <!-- TMPL_LOOP Name="itemloop" -->
+                <tr class="<!-- TMPL_VAR NAME="backgroundcolor" -->">
+                    <td>
+                            <!-- TMPL_VAR NAME="barcode" --> 
+                    </td>
+                    <td>
+                    <!-- TMPL_VAR NAME="homebranch" -->
+                    </td>
+                    <td>
+                    <!-- TMPL_VAR NAME="holdingbranch" -->
+                    </td>
+                    <td>
+                    <!-- TMPL_VAR NAME="itemcallnumber" -->
+                    </td>
+                    <td>
+                <!-- TMPL_IF Name="date_due" -->
+                    <span class="checkedout">Due <!-- TMPL_VAR NAME="date_due" --></span>
+                <!-- /TMPL_IF -->
+            
+                <!-- TMPL_IF NAME="available" -->
+                    <label>Hold this item 
+                        <input type="checkbox" name="checkitem" value="<!-- TMPL_VAR NAME="itemnumber" -->" />
+                </label>     
+                
+                <!-- /TMPL_IF -->
+                    
+                <!-- TMPL_IF name="reservedate" -->
+                    Item on hold since <!-- TMPL_VAR NAME="reservedate" -->
+                <!-- /TMPL_IF -->
+            
+                <!-- TMPL_IF Name="message" -->
+                    Cant' be selected (lost or long overdue)
+                <!-- /TMPL_IF -->
+            
+                <!-- TMPL_IF Name="notforloan" -->
+                    This item is not for loan (<!-- TMPL_VAR NAME="notforloanvalue" -->)
+                <!-- /TMPL_IF -->
+            
+                <!-- TMPL_IF Name="transfertwhen" -->
+                    Item in transit from <!-- TMPL_VAR NAME="transfertfrom" -->,
+                    to <!-- TMPL_VAR NAME="transfertto" -->, on <!-- TMPL_VAR NAME="transfertwhen" -->
+                <!-- /TMPL_IF -->
+                    </td>
+                    <td>
+                    <!-- TMPL_IF NAME="reservedate"-->
+                                Item on hold
+                    <!-- TMPL_ELSE -->
+                            No holds
+                    <!-- /TMPL_IF -->
+                    </td>
+                </tr>
+            <!-- /TMPL_LOOP --> <!-- itemloop -->
+        </table>
+        <p> <label for="requestany" style="float: none; width: auto; padding-left : 1em;">Hold next available copy </label>
+            <input type="checkbox" id="requestany" name="request" checked="checked" value="Any" />
         </p>
-    <h4>
-     ...Or Choose Acceptable Formats
-    </h4>
-
-<!-- TMPL_LOOP NAME="TYPE_ROWS" -->
-<ul><!-- TMPL_LOOP NAME="ROW" -->
-<li><input type="checkbox" name="itemtype" value="<!-- TMPL_VAR NAME="itemtype" -->" class="checkbox"><!-- TMPL_VAR NAME="description" --></li>
-<!-- /TMPL_LOOP -->
-</ul><!-- /TMPL_LOOP -->
-<!-- <table>
-    <tr>
-    <th>Number of Copies</th><th>Requests already received for this title</th> <th>Items curently on issue, due back</th></tr>
-    <td><!-- TMPL_VAR NAME="itemcount" --></td>
-    <td><!-- TMPL_VAR NAME="reservecount" --></td>
-     <td><!-- TMPL_LOOP NAME="ITEMS" --><!-- TMPL_VAR NAME="date_due" -->, <!-- /TMPL_LOOP --></td>
-</table> -->
-</div>
-<p><input type="submit" class="submit" value="Confirm Reservation &gt;&gt;"></p>
-</form>
-<!-- /TMPL_IF -->
-
-<!-- end of the first form -->
+        <input type="hidden" name="biblioitem" value="<!-- TMPL_VAR NAME="biblioitemnumber" -->" />
+        <!-- /TMPL_LOOP --> <!-- bibitemloop -->
+        <p>Pick up from:<select name="branch" id="branch">
+                    <option>-- Please choose your branch --</option>
+                    <!-- TMPL_LOOP NAME="CGIbranch" --><option value="<!-- TMPL_VAR NAME="value" -->"><!-- TMPL_VAR NAME="branch" --></option><!-- /TMPL_LOOP -->
+            </select>
+        </p>
+        <input type="submit" value="Place Hold" />
+        <input type="hidden" name="all" value="1"/></p>
+        <input type="hidden" name="place_reserve" value="1"/>
+        </form>
+    <!-- end of the first form -->
 <!-- /TMPL_IF -->
 
 <!-- TMPL_IF NAME="item_types_selected" -->
index 4c2794d..2b59c46 100755 (executable)
@@ -28,7 +28,6 @@ use C4::Date;
 use C4::Context;
 use C4::Members;
 use C4::Branch; # GetBranches
-use Data::Dumper;
 
 my $MAXIMUM_NUMBER_OF_RESERVES = C4::Context->preference("maxreserves");
 
@@ -143,7 +142,6 @@ $template->param( itemcount => $itemcount );
 my %types;
 my %itemtypes;
 my @duedates;
-#die @items;
 foreach my $itm (@items) {
     push @duedates, { date_due => format_date( $itm->{'date_due'} ) }
       if defined $itm->{'date_due'};
@@ -195,71 +193,40 @@ $template->param( TYPE_ROWS => \@typerows );
 $width = 2 * $width - 1;
 $template->param( totalwidth => 2 * $width - 1, );
 
-if ( $query->param('item_types_selected') ) {
-
-    # this is what happens after the itemtypes have been selected. Stage 2
-    my @itemtypes = $query->param('itemtype');
-    my $fee       = 0;
-    my $proceed   = 0;
-    if (@itemtypes) {
-        my %newtypes;
-        foreach my $itmtype (@itemtypes) {
-            $newtypes{$itmtype} = $itemtypes{$itmtype};
-        }
-        my @types = values %newtypes;
-        $template->param( TYPES => \@types );
-        foreach my $type (@itemtypes) {
-            my @reqbibs;
-            foreach my $item (@items) {
-                if ( $item->{'itemtype'} eq $type ) {
-                    push @reqbibs, $item->{'biblioitemnumber'};
-                }
-            }
-            $fee +=
-              GetReserveFee( undef, $borrowernumber, $biblionumber, 'o',
-                \@reqbibs );
+if ( $query->param('place_reserve') ) {
+    my @bibitems=$query->param('biblioitem');
+    my $notes=$query->param('notes');
+    my $checkitem=$query->param('checkitem');
+    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 ($checkitem ne ''){
+        $rank = '0';
+        my $item = $checkitem;
+        $item = GetItem($item);
+        if ( $item->{'holdingbranch'} eq $branch ){
+            $found = 'W' unless C4::Context->preference('ReservesNeedReturns');
         }
-        $proceed = 1;
-    }
-    elsif ( $query->param('all') ) {
-        $template->param( all => 1 );
-        # No idea why fee would be set to 1 ... it's supposed to be a monetary value, not a flag
-               # -- JF
-               #$fee = 1;
-               $proceed = 1;
-    }
-    if ( $proceed && $branch ) {
-        $fee = sprintf "%.02f", $fee;
-               if ($fee > 1) {
-        $template->param( fee => $fee, istherefee => $fee > 0 ? 1 : 0 );
-               }
-        $template->param( item_types_selected => 1 );
-        $template->param( no_branch_selected => 1 ) unless ( $branch != '' );
     }
-    else {
-        $template->param( message            => 1 );
-        $template->param( no_items_selected  => 1 ) unless ($proceed);
-        $template->param( no_branch_selected => 1 ) unless ($branch);
-    }
-}
-elsif ( $query->param('place_reserve') ) {
-
-    # here we actually do the reserveration. Stage 3.
-    my $title     = $bibdata->{'title'};
-    my @itemtypes = $query->param('itemtype');
-    foreach my $type (@itemtypes) {
-        my @reqbibs;
-        foreach my $item (@items) {
-            if ( $item->{'itemtype'} eq $type ) {
-                push @reqbibs, $item->{'biblioitemnumber'};
-            }
+        
+        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++;
         }
-        AddReserve( $branch, $borrowernumber, $biblionumber, 'o',
-            \@reqbibs, $rank, '', $title );
     }
-    if ( $query->param('all') ) {
-        AddReserve( $branch, $borrowernumber, $biblionumber, 'a',
-            undef, $rank, '', $title );
+    # here we actually do the reserveration. Stage 3.
+    if ($query->param('request') eq 'any'){
+        # place a request on 1st available
+        AddReserve($branch,$borrowernumber,$biblionumber,'a',\@realbi,$rank,$notes,$bibdata->{'title'},$checkitem,$found);
+    } else {
+        AddReserve($branch,$borrowernumber,$biblionumber,'a',\@realbi,$rank,$notes,$bibdata->{'title'},$checkitem, $found);
     }
     print $query->redirect("/cgi-bin/koha/opac-user.pl");
 }
@@ -316,6 +283,139 @@ else {
     }
 }
 
+
+my @branchcodes;
+my %itemnumbers_of_biblioitem;
+my @itemnumbers  = @{ get_itemnumbers_of($biblionumber)->{$biblionumber} };
+my $iteminfos_of = GetItemInfosOf(@itemnumbers);
+
+foreach my $itemnumber (@itemnumbers) {
+    push( @branchcodes,
+        $iteminfos_of->{$itemnumber}->{homebranch},
+        $iteminfos_of->{$itemnumber}->{holdingbranch} );
+
+    my $biblioitemnumber = $iteminfos_of->{$itemnumber}->{biblioitemnumber};
+    push( @{ $itemnumbers_of_biblioitem{$biblioitemnumber} }, $itemnumber );
+}
+
+# @branchcodes = uniq @branchcodes;
+
+my @biblioitemnumbers = keys %itemnumbers_of_biblioitem;
+
+my $branchinfos_of      = get_branchinfos_of(@branchcodes);
+my $notforloan_label_of = get_notforloan_label_of();
+my $biblioiteminfos_of  = GetBiblioItemInfosOf(@biblioitemnumbers);
+
+my @itemtypes;
+foreach my $biblioitemnumber (@biblioitemnumbers) {
+    push @itemtypes, $biblioiteminfos_of->{$biblioitemnumber}{itemtype};
+}
+
+my $itemtypeinfos_of = get_itemtypeinfos_of(@itemtypes);
+
+my @bibitemloop;
+
+foreach my $biblioitemnumber (@biblioitemnumbers) {
+    my $biblioitem = $biblioiteminfos_of->{$biblioitemnumber};
+
+    $biblioitem->{description} =
+      $itemtypeinfos_of->{ $biblioitem->{itemtype} }{description};
+
+    foreach
+      my $itemnumber ( @{ $itemnumbers_of_biblioitem{$biblioitemnumber} } )
+    {
+        my $item = $iteminfos_of->{$itemnumber};
+
+        $item->{homebranchname} =
+          $branchinfos_of->{ $item->{homebranch} }{branchname};
+
+        # if the holdingbranch is different than the homebranch, we show the
+        # holdingbranch of the document too
+        if ( $item->{homebranch} ne $item->{holdingbranch} ) {
+            $item->{holdingbranchname} =
+              $branchinfos_of->{ $item->{holdingbranch} }{branchname};
+        }
+        
+#      add information
+       $item->{itemcallnumber} = $item->{itemcallnumber};
+       
+        # if the item is currently on loan, we display its return date and
+        # change the background color
+        my $issues= GetItemIssue($itemnumber);
+        if ( $issues->{'date_due'} ) {
+            $item->{date_due} = format_date($issues->{'date_due'});
+            $item->{backgroundcolor} = 'onloan';
+        }
+
+        # checking reserve
+        my ($reservedate,$reservedfor,$expectedAt) = GetReservesFromItemnumber($itemnumber);
+        my $ItemBorrowerReserveInfo = GetMemberDetails( $reservedfor, 0);
+
+        if ( defined $reservedate ) {
+            $item->{backgroundcolor} = 'reserved';
+            $item->{reservedate}     = format_date($reservedate);
+            $item->{ReservedForBorrowernumber}     = $reservedfor;
+            $item->{ReservedForSurname}     = $ItemBorrowerReserveInfo->{'surname'};
+            $item->{ReservedForFirstname}     = $ItemBorrowerReserveInfo->{'firstname'};
+            $item->{ExpectedAtLibrary}     = $expectedAt;
+            
+        }
+
+        # Management of the notforloan document
+        if ( $item->{notforloan} ) {
+            $item->{backgroundcolor} = 'other';
+            $item->{notforloanvalue} =
+              $notforloan_label_of->{ $item->{notforloan} };
+        }
+
+        # Management of lost or long overdue items
+        if ( $item->{itemlost} ) {
+
+            # FIXME localized strings should never be in Perl code
+            $item->{message} =
+                $item->{itemlost} == 1 ? "(lost)"
+              : $item->{itemlost} == 2 ? "(long overdue)"
+              : "";
+            $item->{backgroundcolor} = 'other';
+        }
+
+        # Check of the transfered documents
+        my ( $transfertwhen, $transfertfrom, $transfertto ) =
+          GetTransfers($itemnumber);
+
+        if ( $transfertwhen ne '' ) {
+            $item->{transfertwhen} = format_date($transfertwhen);
+            $item->{transfertfrom} =
+              $branchinfos_of->{$transfertfrom}{branchname};
+            $item->{transfertto} = $branchinfos_of->{$transfertto}{branchname};
+               $item->{nocancel} = 1;
+        }
+
+        # If there is no loan, return and transfer, we show a checkbox.
+        $item->{notforloan} = $item->{notforloan} || 0;
+
+        # An item is available only if:
+        if (
+            not defined $reservedate    # not reserved yet
+            and $issues->{'date_due'} eq ''         # not currently on loan
+            and not $item->{itemlost}   # not lost
+            and not $item->{notforloan} # not forbidden to loan
+            and $transfertwhen eq ''    # not currently on transfert
+          )
+        {
+            $item->{available} = 1;
+        }
+
+        push @{ $biblioitem->{itemloop} }, $item;
+    }
+
+    push @bibitemloop, $biblioitem;
+}
+
+# display infos
+$template->param(
+    bibitemloop       => \@bibitemloop,
+);
 output_html_with_http_headers $query, $cookie, $template->output;
 
 # Local Variables:
index 43e6445..ba5ae75 100755 (executable)
@@ -49,20 +49,16 @@ my $borrowernumber=GetMember($borrower,'cardnumber');
 my $checkitem=$input->param('checkitem');
 my $found;
 
-#new op : 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';
-               my $item = $checkitem;
-               $item = GetItem($item);
-               if ( $item->{'holdingbranch'} eq $branch ){
-               $found = 'W' unless C4::Context->preference('ReservesNeedReturns');
-               }
-
-
+    $rank[0] = '0';
+    my $item = $checkitem;
+    $item = GetItem($item);
+    if ( $item->{'holdingbranch'} eq $branch ){
+        $found = 'W' unless C4::Context->preference('ReservesNeedReturns');
+    }
 }
 
-# END of new op .
-
 if ($type eq 'str8' && $borrowernumber ne ''){
        my $count=@bibitems;
        @bibitems=sort @bibitems;
index afed443..d372175 100755 (executable)
@@ -104,8 +104,7 @@ if ($cardnumber) {
     }
 
     # we check the date expiricy of the borrower (only if there is an expiry date, otherwise, set to 1 (warn)
-    warn "BOR : ".$borrowerinfo->{'dateexpiry'};
-    if ($borrowerinfo->{'dateexpiry'}) {
+    if ($borrowerinfo->{'dateexpiry'} ne '0000-00-00') {
         my $warning = (Date_to_Days(split /-/,$date) > Date_to_Days( split /-/,$borrowerinfo->{'dateexpiry'}));
         if ( $warning > 0 ) {
             $expiry = 1;
@@ -221,7 +220,7 @@ foreach my $itemnumber (@itemnumbers) {
     push( @{ $itemnumbers_of_biblioitem{$biblioitemnumber} }, $itemnumber );
 }
 
-@branchcodes = uniq @branchcodes;
+@branchcodes = uniq @branchcodes;
 
 my @biblioitemnumbers = keys %itemnumbers_of_biblioitem;