I've update Jquery to 1.7 version (#1067290: Fix jQuery 1.7 for Drupal 6) and faced a little problem in Drupal.dhtmlMenu.cookieSet() function.

Drupal.dhtmlMenu.cookieSet = function() {
  var expanded = new Array();
  $('li.expanded').each(function() {
    expanded.push($(this).find('a:first').attr('id').substr(5));
  });
  document.cookie = 'dhtml_menu=' + expanded.join(',') + ';path=/';
}

In 1.3 version of Jquery .attr('id') returned empty string if id wasn't defined. But in recent versions it returns undefined.
So we should check the return before call .substr() method.

CommentFileSizeAuthor
#1 jquery1.7_capability-1364882-1.patch581 bytesLeksat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Leksat’s picture

Status: Active » Needs review
FileSize
581 bytes
vuil’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)

I closed the issue as Closed (outdated) because it is for unsupported 6.x version of Drupal.

Thanks to everyone for your contribution time!