In dhtml_menu.module at line 253, there was a fix in the last version that goes like this:

    // pid is going to be a DOM identifier, and can't have some characters
    $forbid = array(' ', '(', ')', '{', '}', '[', ']');
    $pid = str_replace($forbid, '', $pid);

It works for those characters but misses a lot of them. Here is a better fix:

    // pid is going to be a DOM identifier, and can't have some characters
    $pid = ereg_replace('[^A-Za-z0-9]', '', $pid);

It works for me. I'd love it to be in the next release.

Thanks :-)

CommentFileSizeAuthor
#3 dhtml_menu-clean-id-222303-3.patch707 bytescburschka

Comments

brmassa’s picture

Status: Active » Fixed

Martin,

very good patch. thanks.

regards,

massa

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

cburschka’s picture

Status: Closed (fixed) » Reviewed & tested by the community
StatusFileSize
new707 bytes

Patch must be committed to HEAD before DRUPAL-6--1. Here is a port for HEAD.

cburschka’s picture

Status: Reviewed & tested by the community » Fixed

Committed.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.