Attachment 33222 Details for Bug 12892 – Bug 12892 : Alternate fix to restore the deleted code

[patch] Bug 12892 : Alternate fix to restore the deleted code

Bug-12892--Alternate-fix-to-restore-the-deleted-co.patch (text/plain), 2.72 KB, created by Chris Cormack on 2014-11-05 07:22:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Cormack

Created: 2014-11-05 07:22:15 UTC

Size: 2.72 KB

patch

obsolete

>From 66ea388f91d8c9c869e36360d1f151fbaecd31fd Mon Sep 17 00:00:00 2001
>From: Chris Cormack <chris@bigballofwax.co.nz>
>Date: Wed, 5 Nov 2014 20:00:39 +1300
>Subject: [PATCH] Bug 12892 : Alternate fix to restore the deleted code
>
>This doesn't depend on anything other patch.
>
>1. Create a reservation for a Borrower with pickup branch
>2. Check-in the corresponding Item in the pickup branch and confirm the hold
>   for the Borrower
>3. Go to circ/circulation.pl for the Borrower and verify that the reservation
>   is "Waiting for pickup"
>4. In the Borrower -> Check Out -tab (circ/circulation.pl) on the right of the
>   big "Checking out to borrowername" should be a notification about holds ready
>   for pickup.
>---
> circ/circulation.pl |   29 +++++++++++++++++++++++++++--
> 1 file changed, 27 insertions(+), 2 deletions(-)
>
>diff --git a/circ/circulation.pl b/circ/circulation.pl
>index 2a1f211..2cf88b8 100755
>--- a/circ/circulation.pl
>+++ b/circ/circulation.pl
>@@ -390,8 +390,33 @@ if ($borrowernumber) {
>     $template->param(
>         holds_count => Koha::Database->new()->schema()->resultset('Reserve')
>           ->count( { borrowernumber => $borrowernumber } ) );
>-
>-    $template->param( adultborrower => 1 ) if ( $borrower->{'category_type'} eq 'A' );
>+    my @borrowerreserv = GetReservesFromBorrowernumber($borrowernumber);
>+
>+    my @WaitingReserveLoop;
>+    foreach my $num_res (@borrowerreserv) {
>+        if ( $num_res->{'found'} && $num_res->{'found'} eq 'W' ) {
>+            my $getiteminfo  = GetBiblioFromItemNumber( $num_res->{'itemnumber'} );
>+            my $itemtypeinfo = getitemtypeinfo( (C4::Context->preference('item-level_itypes')) ? $getiteminfo->{'itype'} : $getiteminfo->{'itemtype'} );
>+            my %getWaitingReserveInfo;
>+            $getWaitingReserveInfo{title} = $getiteminfo->{'title'};
>+            $getWaitingReserveInfo{biblionumber} =
>+              $getiteminfo->{'biblionumber'};
>+            $getWaitingReserveInfo{itemtype} = $itemtypeinfo->{'description'};
>+            $getWaitingReserveInfo{author}   = $getiteminfo->{'author'};
>+            $getWaitingReserveInfo{itemcallnumber} =
>+              $getiteminfo->{'itemcallnumber'};
>+            $getWaitingReserveInfo{reservedate} =
>+              format_date( $num_res->{'reservedate'} );
>+            $getWaitingReserveInfo{waitingat} =
>+              GetBranchName( $num_res->{'branchcode'} );
>+            $getWaitingReserveInfo{waitinghere} = 1
>+              if $num_res->{'branchcode'} eq $branch;
>+            push( @WaitingReserveLoop, \%getWaitingReserveInfo );
>+        }
>+    }
>+    $template->param( WaitingReserveLoop => \@WaitingReserveLoop );
>+    $template->param( adultborrower => 1 )
>+      if ( $borrower->{'category_type'} eq 'A' );
> }
> 
> my @values;
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 12892: 31471 | 31472 | 31581 | 31600 | 31601 | 31603 | 31604 | 31605 | 31607 | 31608 | 31670 | 31671 | 31672 | 31962 | 31963 | 32010 | 32017 | 32044 | 32057 | 32066 | 32069 | 33222 | 33225 | 33236 | 33275