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

Bug 24621: Phase out jquery.cookie.js: Basic MARC editor 

This patch removes the use of the jquery.cookie plugin by the basic MARC
editor for setting two interface preferences: Showing MARC tag numbers
and showing MARC documentation links.

To test, apply the patch and open the basic MARC editor.

 - From the "Settings" menu, select the "Show MARC tag documentation
   links" link. This should toggle the display of the "?" link next to
   MARC tag descriptions.
 - Reload the page to confirm that your selection has been saved.
 - Perform the same test with the "Show tags" menu item.

You can also follow the changes to the cookie using the browser's
developer console. Look for cookies named 'marcdocs_***' and
'marctags_***' (where *** is the borrowernumber of the logged-in
user) and confirm that the value of each flips from "show" to "hide" and
back.

https://developer.mozilla.org/en-US/docs/Tools/Storage_Inspector
https://developers.google.com/web/tools/chrome-devtools/storage/cookies

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
This commit is contained in:
Owen Leonard 2020-02-10 19:04:57 +00:00 committed by Joy Nelson
parent fb45f5bb98
commit a2fe9fa991
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt
View File

@ -39,14 +39,14 @@
});
/* check cookie to hide/show marcdocs*/
if($.cookie("marcdocs_[% borrowernumber | html %]") == 'hide'){
if( Cookies.get("marcdocs_[% borrowernumber | html %]") == 'hide'){
toggleMARCdocLinks(false);
} else {
toggleMARCdocLinks(true);
}
$("#marcDocsSelect").click(function(){
if($.cookie("marcdocs_[% borrowernumber | html %]") == 'hide'){
if( Cookies.get("marcdocs_[% borrowernumber | html %]") == 'hide'){
toggleMARCdocLinks(true);
} else {
toggleMARCdocLinks(false);
@ -54,7 +54,7 @@
});
/* check cookie to hide/show marc tags*/
var marctags_cookie = $.cookie("marctags_[% borrowernumber | html %]");
var marctags_cookie = Cookies.get("marctags_[% borrowernumber | html %]");
if( marctags_cookie == 'hide'){
toggleMARCTagLinks(false);
} else if( marctags_cookie == 'show'){
@ -68,7 +68,7 @@
}
$("#marcTagsSelect").click(function(){
if( $.cookie("marctags_[% borrowernumber | html %]") == 'hide'){
if( Cookies.get("marctags_[% borrowernumber | html %]") == 'hide'){
toggleMARCTagLinks(true)
} else {
toggleMARCTagLinks(false);
@ -117,7 +117,7 @@
if ( !confirm( breedingid ? _("This record cannot be transferred to the advanced editor. Continue?") : _("Any changes will not be saved. Continue?") ) ) return false;
$.cookie( 'catalogue_editor_[% logged_in_user.borrowernumber | html %]', 'advanced', { expires: 365, path: '/' } );
Cookies.set( 'catalogue_editor_[% logged_in_user.borrowernumber | html %]', 'advanced', { expires: 365, path: '/' } );
var biblionumber = [% biblionumber || "null" | html %];
@ -252,11 +252,11 @@
function toggleMARCdocLinks(flag){
if( flag === true ){
$(".marcdocs").show();
$.cookie("marcdocs_[% borrowernumber | html %]",'show', { path: "/", expires: 365 });
Cookies.set("marcdocs_[% borrowernumber | html %]",'show', { path: "/", expires: 365 });
$("#marcDocsSelect i").addClass('fa-check-square-o').removeClass('fa-square-o');
} else {
$(".marcdocs").hide();
$.cookie("marcdocs_[% borrowernumber | html %]",'hide', { path: "/", expires: 365 });
Cookies.set("marcdocs_[% borrowernumber | html %]",'hide', { path: "/", expires: 365 });
$("#marcDocsSelect i").removeClass('fa-check-square-o').addClass('fa-square-o');
}
}
@ -265,12 +265,12 @@
if( flag === true ){
$(".tagnum").show();
$(".subfieldcode").show();
$.cookie("marctags_[% borrowernumber | html %]",'show', { path: "/", expires: 365 });
Cookies.set("marctags_[% borrowernumber | html %]",'show', { path: "/", expires: 365 });
$("#marcTagsSelect i").addClass('fa-check-square-o').removeClass('fa-square-o');
} else {
$(".tagnum").hide();
$(".subfieldcode").hide();
$.cookie("marctags_[% borrowernumber | html %]",'hide', { path: "/", expires: 365 });
Cookies.set("marctags_[% borrowernumber | html %]",'hide', { path: "/", expires: 365 });
$("#marcTagsSelect i").removeClass('fa-check-square-o').addClass('fa-square-o');
}
}