[TOMAHAWK-1027] JScookmenu is generating malformed HTML when usign ThemeOffice theme

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.1.6
• Fix Version/s: 1.1.7
• Component/s: JS Cook Menu
• Environment: Mozilla Firefox 2.0
• Assignee: Cagatay Civici
• Reporter: Lucas Machado
• Created: 18/Jun/07 14:20
• Updated: 03/Sep/08 04:46
• Resolved: 28/Jun/07 08:14

Description

When usign JS cook menu with ThemeOffice the default implementation generates malformed HTML when the menu have sub-menu itens.

In the generated HTML is missing some </td>

I fix this problem changing the line 195 of JSCookMenu.js from
str += '<td class="' + classStr + 'Text">' + item[1];
to
str += '</td><td class="' + classStr + 'Text">' + item[1] +"</td>";

I did not test it on all possible cenarios. but this change seems to perform well.

This is specially bad when using facelets that require XHTML. As the XHTML is mandatory to be a wellformed HTML Firefox halts the execution of the script when it tries to print the result
on line 321 obj.innerHTML = str;
Apparently Internet Explorer doesn´t check HTML generated by Javascript and the menu works fine even on facelets.