[Koha-bugs] [Bug 12320] New: $.cookie('foo', null) deprecated

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12320

            Bug ID: 12320
           Summary: $.cookie('foo', null) deprecated
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: minor
          Priority: P5 - low
         Component: Architecture, internals, and plumbing
          Assignee: gmcharlt at gmail.com
          Reporter: gmcharlt at gmail.com
        QA Contact: testopia at bugs.koha-community.org

With the upgrade of jquery-cookie to the latest version (see bug 11369), the
following construct to delete a cookie is no longer supported:

$.cookie('foo', null)

Instead, per
https://github.com/carhartl/jquery-cookie/blob/master/CHANGELOG.md, it should
now be

$.removeCookie('foo');

There is at least one direct use of the deprecated construct:

koha-tmpl/intranet-tmpl/prog/en/js/pages/batchMod.js:   
$.cookie("showColumns",null);

and possibly others:

koha-tmpl/intranet-tmpl/prog/en/js/staff-global.js:    
$.cookie("holdfor",null, { path: "/", expires: 0 });
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt:   
$.cookie("holdfor",null, { path: "/", expires: 0 });
koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt:[% IF (
borrowernumber ) %]if($.cookie("holdfor") != [% borrowernumber %]){
$.cookie("holdfor",null, { path: "/", expires: 0 }); }[% ELSE
%]$.cookie("holdfor",null, { path: "/", expires: 0 });[% END %]

-- 
You are receiving this mail because:
You are watching all bug changes.