From fe4a8d703681624f296d609c3490b649d1bd76c3 Mon Sep 17 00:00:00 2001 From: Mark Tompsett Date: Fri, 24 Apr 2015 10:10:22 -0400 Subject: [PATCH] Bug 14061: Fix broken mapping for not withdrawn items. TEST PLAN --------- 1) Check what values to have entered for WITHDRAWN authorized values (Home -> Koha Administration -> Authorized values change dropdown to WITHDRAWN) 2) Add a new authorized value of 0 () if one is missing. 3) Add a new authorized value of 1 (Withdrawn) if one is missing. 4) Run inventory on a filtered set such that the selected barcode is not withdrawn. (Home -> Tools -> Inventory/stocktaking) -- the Withdrawn column displays '0'. 5) Run inventory on a filtered set such that the selected barcode is withdrawn. -- the Withdrawn column displays '1'. 6) Apply patch 7) Rerun steps 4 & 5 -- the Withdrawn column displays the descriptions as seen in steps 1-3. 8) run koha qa test tools --- C4/Items.pm | 2 +- tools/inventory.pl | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/C4/Items.pm b/C4/Items.pm index f443d38..dd5504b 100644 --- a/C4/Items.pm +++ b/C4/Items.pm @@ -1126,7 +1126,7 @@ sub GetItemsForInventory { foreach (keys %$row) { # If the koha field is mapped to a marc field my ($f, $sf) = GetMarcFromKohaField("items.$_", $row->{'frameworkcode'}); - if ($f and $sf) { + if (defined $f and defined $sf) { # We replace the code with it's description my $authvals = C4::Koha::GetKohaAuthorisedValuesFromField($f, $sf, $row->{'frameworkcode'}); $row->{$_} = $authvals->{$row->{$_}} if defined $authvals->{$row->{$_}}; diff --git a/tools/inventory.pl b/tools/inventory.pl index b8cc31d..91e1c72 100755 --- a/tools/inventory.pl +++ b/tools/inventory.pl @@ -272,7 +272,7 @@ foreach my $item ( @scanned_items ) { # If the koha field is mapped to a marc field my $fc = $item->{'frameworkcode'} || ''; my ($f, $sf) = GetMarcFromKohaField("items.$field", $fc); - if ($f and $sf) { + if (defined $f and defined $sf) { # We replace the code with it's description my $authvals = C4::Koha::GetKohaAuthorisedValuesFromField($f, $sf, $fc); if ($authvals and defined $item->{$field} and defined $authvals->{$item->{$field}}) { -- 1.9.1