From 50db7afeb43171bdde2e2f4e421acc45771ec378 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 21 May 2021 09:09:53 +0200 Subject: [PATCH] Bug 28409: Simplify data validation Simplify the affectation then trust it. Signed-off-by: Victor Grousset/tuxayo JD Amended patch: remove duplicate comma Signed-off-by: Kyle M Hall Signed-off-by: Nick Clemens --- opac/opac-shelves.pl | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/opac/opac-shelves.pl b/opac/opac-shelves.pl index e3b167799d..0ec193c9ec 100755 --- a/opac/opac-shelves.pl +++ b/opac/opac-shelves.pl @@ -80,13 +80,7 @@ if (C4::Context->preference("BakerTaylorEnabled")) { my $referer = $query->param('referer') || $op; my $category = 1; -if ( $query->param('category') && ( - ($query->param('category') == 1) || - ($query->param('category') == 2) - ) - ){ - $category = $query->param('category'); -} +$category = 2 if $query->param('category') && $query->param('category') == 2; my ( $shelf, $shelfnumber, @messages ); @@ -115,7 +109,7 @@ if ( $op eq 'add_form' ) { $shelf = Koha::Virtualshelf->new( { shelfname => scalar $query->param('shelfname'), sortfield => scalar $query->param('sortfield'), - category => $category || 1, + category => $category, allow_change_from_owner => $allow_changes_from > 0, allow_change_from_others => $allow_changes_from == ANYONE, owner => scalar $loggedinuser, @@ -442,7 +436,7 @@ $template->param( referer => $referer, shelf => $shelf, messages => \@messages, - category => ($category == 1 || $category == 2) ? $category : "", + category => $category, print => scalar $query->param('print') || 0, listsview => 1, ); -- 2.39.5