Bugzilla – Attachment 38493 Details for
Bug 14061
Inventory fails to map withdrawn authorized values
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14061: Fix broken mapping for not withdrawn items.
Bug-14061-Fix-broken-mapping-for-not-withdrawn-ite.patch (text/plain), 2.37 KB, created by
Mark Tompsett
on 2015-04-24 14:44:58 UTC
(
hide
)
Description:
Bug 14061: Fix broken mapping for not withdrawn items.
Filename:
MIME Type:
Creator:
Mark Tompsett
Created:
2015-04-24 14:44:58 UTC
Size:
2.37 KB
patch
obsolete
>From fe4a8d703681624f296d609c3490b649d1bd76c3 Mon Sep 17 00:00:00 2001 >From: Mark Tompsett <mtompset@hotmail.com> >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
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 14061
:
38485
|
38486
|
38487
|
38488
|
38489
|
38490
|
38491
|
38493
|
38505
|
38735