Bug 25027: Use localStorage instead of sessionStorage for results browser
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 31 Mar 2020 14:39:12 +0000 (16:39 +0200)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Tue, 14 Apr 2020 15:13:38 +0000 (16:13 +0100)
commit51dbb84a78e7ad833ab2ce6104ed54954ded6835
tree71d0644a2d8ad92d59d38ef6696038e9b5c73dad
parent30ea935720f8848be8a9a467d9dda89275d4557b
Bug 25027: Use localStorage instead of sessionStorage for results browser

Staff side, when a search a done and a result clicked, a browser appears
on the left, to navigate between the different results.

We use sessionStorage to know the list of biblionumber from the result.

As sessionStorage is only for the current tab, we do some ugly things,
to catch the click events, then open the new tab, attach it to the
current window, and put the focus back on the result list.

We really should not do that, and let the user decide what they want to
do with their clicks!

To do so, let use the correct storage, localStorage, and have the
results shared between the windows.

We may need to clear that at some point, isn't it?

Test plan:
Launch a search, click result (left or middle), confirm you see the
browser and that the window/tab opened like any other websites
(depending on your web browser settings).

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/js/browser.js