From 931f3c3f8e333a3dc85936e235e5cdd6635af8e8 Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Sun, 8 Feb 2015 21:57:26 +0100 Subject: [PATCH] Bug 13648: OPAC = hidden removes pull down from edit item form If you uncheck the checkbox OPAC in the bibliographic framework definition for an item subfield that is linked to an authorized value list, the pull down will disappear from the item edit form on staff. The label still shows. 1) Go to administration > bibliographic frameworks 2) Select a framework and go to the item field (MARC21: 952) 3) Edit a subfield with an authorized values list like damaged, lost or withdrawn 4) Uncheck the checkbox for OPAC visibility 5) Edit or add an item within your chosen framework 6) Verify the subfield code and description are shown, but the pull down has disappeared 7) Apply patch, verify the pull down shows now 8) Also test following hidden combination works correctly: OPAC: checked All other options: unchecked Documentation of hidden values: http://wiki.koha-community.org/wiki/Hidden_values With special thanks to Jonathan Druart for helping me figure this out! Signed-off-by: Nicole C. Engard Signed-off-by: Jonathan Druart --- cataloguing/additem.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cataloguing/additem.pl b/cataloguing/additem.pl index 9c7fb3f..f731a77 100755 --- a/cataloguing/additem.pl +++ b/cataloguing/additem.pl @@ -136,7 +136,7 @@ sub generate_subfield_form { $value =~ s/DD/$day/g; } - $subfield_data{visibility} = "display:none;" if (($subfieldlib->{hidden} > 4) || ($subfieldlib->{hidden} < -4)); + $subfield_data{visibility} = "display:none;" if (($subfieldlib->{hidden} > 4) || ($subfieldlib->{hidden} <= -4)); my $pref_itemcallnumber = C4::Context->preference('itemcallnumber'); if (!$value && $subfieldlib->{kohafield} eq 'items.itemcallnumber' && $pref_itemcallnumber) { @@ -231,7 +231,7 @@ sub generate_subfield_form { } } - if ($subfieldlib->{'hidden'}) { + if ( $subfieldlib->{hidden} > 4 or $subfieldlib->{hidden} <= -4 ) { $subfield_data{marc_value} = qq( $authorised_lib{$value}); } else { -- 2.1.0