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 :-)
Comments
Comment #1
brmassa commentedMartin,
very good patch. thanks.
regards,
massa
Comment #2
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #3
cburschkaPatch must be committed to HEAD before DRUPAL-6--1. Here is a port for HEAD.
Comment #4
cburschkaCommitted.
Comment #5
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.