From a2fe9fa991bc074c4703729b0f459a40a2cc0eb4 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Mon, 10 Feb 2020 19:04:57 +0000 Subject: [PATCH] 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 Signed-off-by: Katrin Fischer Signed-off-by: Martin Renvoize Signed-off-by: Joy Nelson --- .../prog/en/modules/cataloguing/addbiblio.tt | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt index 6b0b89a825..7e3d3eaee2 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt @@ -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'); } } -- 2.39.5