Bug 21036: Fix uninitialized value within @itemnumber in string ne
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Wed, 4 Jul 2018 14:49:38 +0000 (16:49 +0200)
committerNick Clemens <nick@bywatersolutions.com>
Tue, 7 May 2019 11:39:24 +0000 (11:39 +0000)
Line from 16.11 log:
Use of uninitialized value within @itemnumber in string ne at /usr/share/koha/prodclone/reserve/modrequest.pl line 70.

Test plan:
Read the change. Not 100% identical (numeric zero) but should be enough.
This line is probably not needed at all.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

reserve/modrequest.pl

index 6b4061c..0d1b2d0 100755 (executable)
@@ -66,7 +66,7 @@ if ($CancelBorrowerNumber) {
 # 2) Cancel or modify the queue list of reserves (without item linked)
 else {
     for (my $i=0;$i<$count;$i++){
-        undef $itemnumber[$i] unless $itemnumber[$i] ne '';
+        undef $itemnumber[$i] if !$itemnumber[$i];
         ModReserve({
             rank => $rank[$i],
             reserve_id => $reserve_id[$i],