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)
committerChris Cormack <chris@bigballofwax.co.nz>
Sun, 7 Jun 2015 07:55:41 +0000 (19:55 +1200)
commitac78b4aa3f4cb38cbf524f74b5a8a12857e67ee8
treeac9a727c457f05f8960c9bcb60efdd3d3a3fa110
parent433441685b68effebdba5876103c3a08cf62f6ee
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>
(cherry picked from commit 0fa0297d7da7af6a9f4cd82b34ac86018391289f)
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
koha-tmpl/intranet-tmpl/prog/en/js/pages/batchMod.js