]> git.koha-community.org Git - koha.git/commit
Bug 28698: Fix news for all displays in all locations
authorFridolin Somers <fridolin.somers@biblibre.com>
Fri, 9 Jul 2021 08:33:03 +0000 (22:33 -1000)
committerAndrew Fuerste-Henry <andrew@bywatersolutions.com>
Wed, 22 Dec 2021 12:15:01 +0000 (12:15 +0000)
commit33369d2522c23230e9f755466b19f9c032629365
treecc48a5e537d7c45b7b0d4067f01d4f69f260004d
parent2f4c9064acfeac531a04775fed9fbe36fd918481
Bug 28698: Fix news for all displays in all locations

Some news are used for custom text with a specific language, for example "opacheader".
The bug is that in these locations, news for all (staff and opac) are displayed.
This is because GetNewsToDisplay uses SQL :
  opac_news.lang = '' OR opac_news.lang = ?

This patch is a quick fix.
This feature maybe needs a big revamping.

Test plan :
1) Remove any news
2) Create a news with empty 'Display location' and some content
3) Display OPAC interface
4) without patch you see in location of 'opacheader', 'opacnavright' ... the content
   of the news
5) With patch you see content of the news only in news table
6) Create a news for opacheader
7) Check it appears in correct language

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
C4/NewsChannels.pm