Bug 29932: Use localStorage instead of cookie for bibs_selected
This patch replaces the use of the jquery.cookie.js plugin in the "Browse selected records" feature. The feature is modified to use localStorage instead of cookies. To test, apply the patch and go to Administration -> System preferences. - Enable the "BrowseResultSelection" system preference. - Perform a catalog search which will return multiple results. - On the search results page, check the checkbox for a number of titles. - Click the "Browse selected biblios" button in the toolbar. - You should be directed to the detail page for the first title you checked. - Clicking the right-arrow in the sidebar should take you through each one of your selected titles. - When you click the "Results" link in the sidebar you should be returned to the same search results set and the same titles should be checked. _ If you uncheck those titles and reload the page the checkboxes should remain unchecked. - If you log out of the staff interface the "bibs_selected" item in localStorage should be deleted. See, for example, https://developer.mozilla.org/en-US/docs/Tools/Storage_Inspector for information about viewing local storage. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
parent
64025e0e4d
commit
7a7d503faa
1 changed files with 3 additions and 2 deletions
|
@ -205,6 +205,7 @@ function logOut(){
|
||||||
removeLastBorrower();
|
removeLastBorrower();
|
||||||
localStorage.removeItem("sql_reports_activetab");
|
localStorage.removeItem("sql_reports_activetab");
|
||||||
localStorage.removeItem("searches");
|
localStorage.removeItem("searches");
|
||||||
|
localStorage.removeItem("bibs_selected");
|
||||||
}
|
}
|
||||||
|
|
||||||
function openHelp(){
|
function openHelp(){
|
||||||
|
@ -320,11 +321,11 @@ function delBibToContext( bibnum ) {
|
||||||
}
|
}
|
||||||
|
|
||||||
function setContextBiblioNumbers( bibnums ) {
|
function setContextBiblioNumbers( bibnums ) {
|
||||||
$.cookie('bibs_selected', JSON.stringify( bibnums ));
|
localStorage.setItem('bibs_selected', JSON.stringify( bibnums ) );
|
||||||
}
|
}
|
||||||
|
|
||||||
function getContextBiblioNumbers() {
|
function getContextBiblioNumbers() {
|
||||||
var r = $.cookie('bibs_selected');
|
var r = localStorage.getItem('bibs_selected');
|
||||||
if ( r ) {
|
if ( r ) {
|
||||||
return JSON.parse(r);
|
return JSON.parse(r);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue