[Koha-bugs] [Bug 12320] $.cookie('foo', null) deprecated
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Thu May 21 19:49:22 CEST 2015
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12320
Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #38601|0 |1
is obsolete| |
--- Comment #4 from Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> ---
Created attachment 39368
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39368&action=edit
[SIGNED-OFF] Bug 12320: Remove deprecated construct to delete cookie
showColumns
$.cookie('foo', null);
is deprecated and should be replaced with
$.removeCookie('foo');
This patch replaces the occurrences for the "showColumns" cookie.
Before this patch, there was a bug on the batchmod tools.
To reproduce the issue:
1/ Go on the Batch item modification tool
2/ Fill the textarea with barcodes and submit
3/ Click on some column names (to create the cookie)
4/ Click on 'Show all columns" (to set the cookie to null)
5/ Don't submit and repeat steps 1 & 2
6/ You should see a js error:
Error: Syntax error, unrecognized expression: :nth-child
...break;q=a}return s},m.error=function(a){throw new Error("Syntax error,
unrecogni...
Test plan:
Confirm the issue has gone away and there is no regression on the column
selection
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
No js error, no regressions, no errors
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list