Replies

• Zerxer Posts: 13Questions: 0Answers: 0

  January 2010 edited January 2010

  I found the cause of the issue, which is this code here:
  
  [code]$("#filter_active").change(function(){
  pTable.fnFilter($(this).is(':checked') ? ' Active' : '', 7);
  }).change();[/code]
  
  Filtering as  Active is breaking the cookie. When I look at my cookie, it ends abruptly at &nbsp, with no semi-colon even. So the semi-colon is ending the cookie (which, I'm pretty sure that's how document.cookie works with semi-colons, right?). So yeah, seems DataTables needs to escape this or something.
  
  By the way, the reason I'm even doing this is because that column can have either 'Inactive' or 'Active' in it and if I just filter by 'Active', it also matches 'Inactive' (case-insensitive?). And I'm pretty sure I had some type of problem when I tried just using a space without actually typing ' '
• allan Posts: 61,665Questions: 1Answers: 10,095 Site admin

  January 2010

  Hi Zerxer,
  
  Good find - thanks for spotting this and reporting it back to me. :-)
  
  Your are quite right in your analysis. The semi-colon was causing issues since DataTables wasn't correctly encoding it. I'll release a fix for this in 1.6.1, but until then you can replace the two cookie functions with the following to fix this in your local copy:
  
  [code]
  /*
  * Function: _fnCreateCookie
  * Purpose: Create a new cookie with a value to store the state of a table
  * Returns: -
  * Inputs: string:sName - name of the cookie to create
  * string:sValue - the value the cookie should take
  * int:iSecs - duration of the cookie
  */
  function _fnCreateCookie ( sName, sValue, iSecs )
  {
  var date = new Date();
  date.setTime( date.getTime()+(iSecs*1000) );
  
  /*
  * Shocking but true - it would appear IE has major issues with having the path being
  * set to anything but root. We need the cookie to be available based on the path, so we
  * have to append the pathname to the cookie name. Appalling.
  */
  sName += '_'+window.location.pathname.replace(/[\/:]/g,"").toLowerCase();
  
  document.cookie = sName+"="+encodeURIComponent(sValue)+
  "; expires="+date.toGMTString()+"; path=/";
  }
  
  /*
  * Function: _fnReadCookie
  * Purpose: Read an old cookie to get a cookie with an old table state
  * Returns: string: - contents of the cookie - or null if no cookie with that name found
  * Inputs: string:sName - name of the cookie to read
  */
  function _fnReadCookie ( sName )
  {
  var sNameEQ = sName +'_'+ window.location.pathname.replace(/[\/:]/g,"").toLowerCase() + "=";
  var sCookieContents = document.cookie.split(';');
  
  for( var i=0 ; i