From a85bd6bac7df3936d1f00be4ee73d3b7d20dc610 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 10 Feb 2021 07:26:15 +0100 Subject: [PATCH] Bug 27650: Fix variable passed to the template in opac-main $template->param( + koha_news => @all_koha_news, We must not pass an array, the number of elements of the hash passed to the template may be inconsistent. It's working because of an error earlier in the script: + @all_koha_news = &GetNewsToDisplay($news_lang,$homebranch); GetNewsToDisplay returns an arrayref Test plan: Define at least 2 news to display on the OPAC main page Hit opac-main.pl => All news are displayed Click one => You see the single news you selected Signed-off-by: Fridolin Somers --- opac/opac-main.pl | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/opac/opac-main.pl b/opac/opac-main.pl index f885a8ab48..af0377f483 100755 --- a/opac/opac-main.pl +++ b/opac/opac-main.pl @@ -61,17 +61,17 @@ elsif (C4::Context->userenv and defined $input->param('branch') and length $inpu } my $news_id = $input->param('news_id'); -my @all_koha_news; +my $all_koha_news; if (defined $news_id){ - @all_koha_news = Koha::News->search({ idnew => $news_id, lang => { '!=', 'koha' } }); # get news that is not staff-only news - if( @all_koha_news ) { # we only expect one btw - $template->param( news_item => $all_koha_news[0] ); + $all_koha_news = Koha::News->search({ idnew => $news_id, lang => { '!=', 'koha' } }); # get news that is not staff-only news + if( $all_koha_news->count ) { # we only expect one btw + $template->param( news_item => $all_koha_news->next ); } else { $template->param( single_news_error => 1 ); } } else { - @all_koha_news = &GetNewsToDisplay( $template->lang, $homebranch); + $all_koha_news = &GetNewsToDisplay( $template->lang, $homebranch); } # For dashboard @@ -104,7 +104,7 @@ if ( $patron ) { } $template->param( - koha_news => @all_koha_news, + koha_news => $all_koha_news, branchcode => $homebranch, daily_quote => Koha::Quotes->get_daily_quote(), ); -- 2.30.0