gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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 <fridolin.somers@biblibre.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
Jonathan Druart 2021-02-10 07:26:15 +01:00
parent e5a44214eb
commit ca4ef19a2b
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
opac/opac-main.pl
View File

@ -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(),
);