Bug 12320: Remove deprecated construct to delete cookie showColumns
authorJonathan Druart <jonathan.druart@biblibre.com>
Tue, 28 Apr 2015 10:52:36 +0000 (12:52 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Thu, 4 Jun 2015 13:07:26 +0000 (10:07 -0300)
commit0fa0297d7da7af6a9f4cd82b34ac86018391289f
tree5e9f9aaaf0e7febeab465250cb604c66e1b61fe5
parent04f5e7d4e7db833c18afe27a4dc4fd5b66b41099
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@gmail.com>
No js error, no regressions, no errors

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
koha-tmpl/intranet-tmpl/prog/en/js/pages/batchMod.js