From 0833724afe7cb0d2f2db32c7e87f3df064e6a016 Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Mon, 26 Oct 2020 10:49:49 +0100 Subject: [PATCH] Bug 26032: (follow-up) show 'is new' filter only if items.new_status is used MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Also adds a class 'item-new-status' to this filter to be able to hidde even if items.new_status used. Addition to test plan: 1) Apply on a database without Koha to MARC mapping on items.new_status 2) Check you dont see the 'is new' filter in items search form 3) Apply on a database with Koha to MARC mapping on items.new_status 4) Check you see class 'form-field item-new-status' Signed-off-by: Séverine QUEUNE Signed-off-by: Katrin Fischer Signed-off-by: Jonathan Druart --- catalogue/itemsearch.pl | 4 ++++ .../intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt | 6 ++++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/catalogue/itemsearch.pl b/catalogue/itemsearch.pl index 2ea32156ac..f5f66fe349 100755 --- a/catalogue/itemsearch.pl +++ b/catalogue/itemsearch.pl @@ -102,6 +102,10 @@ my $itemlost_values = $mss->count ? GetAuthorisedValues($mss->next->authorised_v $mss = Koha::MarcSubfieldStructures->search({ frameworkcode => '', kohafield => 'items.withdrawn', authorised_value => [ -and => {'!=' => undef }, {'!=' => ''}] }); my $withdrawn_values = $mss->count ? GetAuthorisedValues($mss->next->authorised_value) : []; +if ( Koha::MarcSubfieldStructures->search( { frameworkcode => '', kohafield => 'items.new_status' } )->count ) { + $template->param( has_new_status => 1 ); +} + if (scalar keys %params > 0) { # Parameters given, it's a search diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt index 06c1ed8723..81ff512cb8 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/itemsearch.tt @@ -90,7 +90,7 @@ [% END %] [% BLOCK form_field_radio_yes_no %] -
+
@@ -195,7 +195,9 @@ (inclusive)
[% INCLUDE form_field_radio_yes_no name="damaged" %] - [% INCLUDE form_field_radio_yes_no name="new_status" %] + [% IF ( has_new_status ) %] + [% INCLUDE form_field_radio_yes_no name="new_status" divclass='item-new-status' %] + [% END %]