Bug 20864: Only set bibs_selected cookie when BrowseResultSelection is activated
authorOwen Leonard <oleonard@myacpl.org>
Mon, 4 Jun 2018 13:52:28 +0000 (13:52 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 22 Jun 2018 20:16:48 +0000 (20:16 +0000)
commitce6ec1e7faef217950e3450788ea444dad26d7c0
treee6c5eeba2184f569e293a408ed4751854c4be465
parent6965c58308f384bf895050be692a1d71e091514d
Bug 20864: Only set bibs_selected cookie when BrowseResultSelection is activated

This patch adds a system preference check around the call to the
JavaScript include which sets the bibs_selected cookie. With
BrowseResultsSelection disabled, the cookie should never be set.

To test you should be using a browser with a tool for inspecting
cookies. View the cookies for your Koha domain under each of these
conditions, clearing cookies between each step:

Without the patch applied:

 - With  BrowseResultSelection enabled, search for any item. Your cookie
   tool should report that you have a bibs_selected cookie stored.
 - With BrowseResultSelection disabled, search for any item. You should
   have a bibs_selected cookie.

With the patch applied:

 - With the BrowseResultSelection system preference enabled, search for
   any item. Your cookie tool should report that you have a
   bibs_selected cookie stored.
 - With BrowseResultSelection disabled, search for any item. You should
   have no bibs_selected cookie.

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

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/includes/js_includes.inc