Attachment 103188 Details for Bug 13547 – Bug 13547: Add item field 3 (Materials specified) to the OPAC holdings table

[patch] Bug 13547: Add item field 3 (Materials specified) to the OPAC holdings table

Bug-13547-Add-item-field-3-Materials-specified-to-.patch (text/plain), 4.02 KB, created by Katrin Fischer on 2020-04-18 01:54:28 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2020-04-18 01:54:28 UTC

Size: 4.02 KB

patch

obsolete

>From 3daa714fdc81b23901e6c69452a7ef6dd78d89e1 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Nazl=C4=B1=20=C3=87etin?= <nazli@devinim.com.tr>
>Date: Thu, 5 Mar 2020 15:02:37 +0000
>Subject: [PATCH] Bug 13547: Add item field 3 (Materials specified) to the OPAC
> holdings table
>
>Test plan:
>
>1- View a record with Materials specified (field 3) data in the opac
>2- Apply patch
>3- Log in to staff client
>4- Home->Administration->Column Settings->OPAC->holdingst
>5- Set item_materials visibility
>6- Refresh OPAC page
>7- Confirm that the materials specified column has been added after the
>   Call number column.
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> admin/columns_settings.yml                              | 3 +++
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt | 5 +++++
> opac/opac-detail.pl                                     | 3 ++-
> 3 files changed, 10 insertions(+), 1 deletion(-)
>
>diff --git a/admin/columns_settings.yml b/admin/columns_settings.yml
>index 19e0bebb3c..ad5db5492c 100644
>--- a/admin/columns_settings.yml
>+++ b/admin/columns_settings.yml
>@@ -1103,6 +1103,9 @@ modules:
>         -
>           columnname: item_callnumber
>         -
>+          columnname: item_materials
>+          is_hidden: 1
>+        -
>           columnname: item_enumchron
>         -
>           columnname: item_url
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>index bf7f7c6a98..f0a63c50ec 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>@@ -1137,6 +1137,8 @@
>         [% IF ( Koha.Preference('OpacLocationOnDetail') == 'column' && itemdata_location ) %]
>                 <th id="item_shelving_location" data-colname="item_shelving_location" class="shelving_location">Shelving location</th>[% END %]
>                 <th id="item_callnumber" data-colname="item_callnumber" class="call_no">Call number</th>
>+        [% IF ( itemdata_materials ) %]
>+                <th id="item_materials" data-colname="item_materials" class="materials">Materials specified</th>[% END %]
>         [% IF ( itemdata_enumchron ) %]
>                 <th id="item_enumchron" data-colname="item_enumchron" class="vol_info">Vol info</th>[% END %]
>         [% IF ( itemdata_uri ) %]
>@@ -1239,6 +1241,9 @@
>                             [% END %]
>                         [% END %]
>                     </td>
>+                    [% IF ( itemdata_materials) %]
>+                        <td class="materials">[% ITEM_RESULT.materials | html %]</td>
>+                    [% END %]
>                     [% IF ( itemdata_enumchron ) %]
>                         <td class="vol_info">
>                             [% IF ITEM_RESULT.enumchron && ITEM_RESULT.serialseq %]
>diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
>index f1100878bd..b3e8dbda63 100755
>--- a/opac/opac-detail.pl
>+++ b/opac/opac-detail.pl
>@@ -698,7 +698,7 @@ if ( not $viewallitems and @items > $max_items_to_display ) {
>         $itm->{'imageurl'}    = getitemtypeimagelocation( 'opac', $itemtypes->{ $itm->{itype} }->{'imageurl'} );
>         $itm->{'description'} = $itemtypes->{ $itm->{itype} }->{translated_description};
>     }
>-    foreach (qw(ccode enumchron copynumber itemnotes location_description uri)) {
>+    foreach (qw(ccode materials enumchron copynumber itemnotes location_description uri)) {
>         $itemfields{$_} = 1 if ($itm->{$_});
>     }
> 
>@@ -780,6 +780,7 @@ if( C4::Context->preference('ArticleRequests') ) {
>                      norequests              => $norequests,
>                      RequestOnOpac           => C4::Context->preference("RequestOnOpac"),
>                      itemdata_ccode          => $itemfields{ccode},
>+                     itemdata_materials      => $itemfields{materials},
>                      itemdata_enumchron      => $itemfields{enumchron},
>                      itemdata_uri            => $itemfields{uri},
>                      itemdata_copynumber     => $itemfields{copynumber},
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 13547: 100199 | 100279 | 102014 | 102015 | 103172 | 103188 | 103189 | 104402 | 104403