Attachment 36265 Details for Bug 13648 – 0001-SIGNED-OFF-Bug-13648-OPAC-hidden-removes-pull-down

[patch] 0001-SIGNED-OFF-Bug-13648-OPAC-hidden-removes-pull-down

0001-SIGNED-OFF-Bug-13648-OPAC-hidden-removes-pull-down-f.patch (text/plain), 2.33 KB, created by Nicole C. Engard on 2015-03-02 08:41:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nicole C. Engard

Created: 2015-03-02 08:41:14 UTC

Size: 2.33 KB

patch

obsolete

>From 65d76656bbd0c427e0f3dde6348625a9be282af3 Mon Sep 17 00:00:00 2001
>From: Katrin Fischer <Katrin.Fischer.83@web.de>
>Date: Sun, 8 Feb 2015 21:57:26 +0100
>Subject: [PATCH] [SIGNED-OFF] 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 <nengard@bywatersolutions.com>
>---
> cataloguing/additem.pl |    4 ++--
> 1 files 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(<input type="hidden" $attributes /> $authorised_lib{$value});
>             }
>             else {
>-- 
>1.7.2.5
>

Actions: View | Diff | Splinter Review

Attachments on bug 13648: 35735 | 35736 | 36265 | 36573