Attachment 102015 Details for Bug 13547 – Bug 13547: (follow-up) Item field 'Materials specified' would be useful to see in OPAC

[patch] Bug 13547: (follow-up) Item field 'Materials specified' would be useful to see in OPAC

Bug-13547-follow-up-Item-field-Materials-specified.patch (text/plain), 2.60 KB, created by Devinim on 2020-03-27 17:13:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Devinim

Created: 2020-03-27 17:13:52 UTC

Size: 2.60 KB

patch

obsolete

>From 546061fb5967b1d663d1371a9d91eab6512aca7d Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Nazl=C4=B1=20=C3=87etin?= <nazli@devinim.com.tr>
>Date: Fri, 27 Mar 2020 17:13:02 +0000
>Subject: [PATCH] Bug 13547: (follow-up) Item field 'Materials specified' would
> be useful to see in OPAC
>
>---
> Koha/Template/Plugin/ColumnsSettings.pm                 | 11 +++++++++++
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt |  4 ++--
> 2 files changed, 13 insertions(+), 2 deletions(-)
>
>diff --git a/Koha/Template/Plugin/ColumnsSettings.pm b/Koha/Template/Plugin/ColumnsSettings.pm
>index 4617969d3af..ee9febe1ce0 100644
>--- a/Koha/Template/Plugin/ColumnsSettings.pm
>+++ b/Koha/Template/Plugin/ColumnsSettings.pm
>@@ -52,4 +52,15 @@ sub GetColumns {
>         : $columns
> }
> 
>+sub is_hidden {
>+    my ( $self, $module, $page, $table, $column_name ) = @_;
>+    my $columns = C4::Utils::DataTables::ColumnsSettings::get_columns( $module, $page, $table );
>+    foreach my $keys(@$columns){
>+        if($keys->{'columnname'} eq $column_name){
>+            return $keys->{'is_hidden'};
>+        }
>+    }
>+    return 0;
>+}
>+
> 1;
>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 e1c0420218f..a06cd7313c2 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>@@ -1137,7 +1137,7 @@
>         [% 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 ) %]
>+        [% UNLESS ColumnsSettings.is_hidden( 'opac', 'biblio-detail', 'holdingst', 'item_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 %]
>@@ -1241,7 +1241,7 @@
>                             [% END %]
>                         [% END %]
>                     </td>
>-                    [% IF ( itemdata_materials) %]
>+                    [% UNLESS ColumnsSettings.is_hidden( 'opac', 'biblio-detail', 'holdingst', 'item_materials') %]
>                         <td class="materials">[% ITEM_RESULT.materials | html %]</td>
>                     [% END %]
>                     [% IF ( itemdata_enumchron ) %]
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

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