Bugzilla – Attachment 37715 Details for
Bug 8474
Holds done by librarians cause silent errors in koha-error_log
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 8474 - Holds done by librarians cause silent errors in koha-error_log, placerequest.pl -> $checkitem
Bug-8474---Holds-done-by-librarians-cause-silent-e.patch (text/plain), 1.64 KB, created by
Jonathan Druart
on 2015-04-13 09:12:51 UTC
(
hide
)
Description:
Bug 8474 - Holds done by librarians cause silent errors in koha-error_log, placerequest.pl -> $checkitem
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2015-04-13 09:12:51 UTC
Size:
1.64 KB
patch
obsolete
>From fd95004804114a48a4686d4db64331f367d312c6 Mon Sep 17 00:00:00 2001 >From: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> >Date: Wed, 23 Jul 2014 11:15:57 +0300 >Subject: [PATCH] Bug 8474 - Holds done by librarians cause silent errors in > koha-error_log, placerequest.pl -> $checkitem > >Squashed this rogue variable initialization > >Test plan: >1/ Place a hold at a biblio level (on the next available item) >2/ Without this patch you get >placerequest.pl: Use of uninitialized value $checkitem in string ne at /home/koha/src/reserve/placerequest.pl line 75. >placerequest.pl: Use of uninitialized value $checkitem in string ne at /home/koha/src/reserve/placerequest.pl line 101. > >Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> >--- > reserve/placerequest.pl | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl >index 1499ea5..5d9339f 100755 >--- a/reserve/placerequest.pl >+++ b/reserve/placerequest.pl >@@ -72,7 +72,7 @@ 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 ''){ >+if (defined $checkitem && $checkitem ne ''){ > $rank[0] = '0' unless C4::Context->preference('ReservesNeedReturns'); > my $item = $checkitem; > $item = GetItem($item); >@@ -98,7 +98,7 @@ if ($type eq 'str8' && $borrower){ > } > my $const; > >- if ($checkitem ne ''){ >+ if (defined $checkitem && $checkitem ne ''){ > my $item = GetItem($checkitem); > if ($item->{'biblionumber'} ne $biblionumber) { > $biblionumber = $item->{'biblionumber'}; >-- >2.1.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 8474
:
11000
|
29964
|
37715
|
37932