Details
-
Bug
-
Resolution: Fixed
-
Major
-
2.1 Final
-
None
-
IE, Firefox
Description
If you don't want to expose a context menu and no context menu items are configured, the tree control produces a javascript error:
'mgnlTreeControlMenu is not defined'
patch for revision 1437 of tagged version /magnolia2.1 :
diff -uBbPr originals/magnolia-2.1/src/main/info/magnolia/cms/gui/control/Tree.java patchedForSendingPatches/magnolia-2.1/src/main/info/magnolia/cms/gui/control/Tree.java
— originals/magnolia-2.1/src/main/info/magnolia/cms/gui/control/Tree.java 2005-08-31 11:29:16.000000000 +0200
+++ patchedForSendingPatches/magnolia-2.1/src/main/info/magnolia/cms/gui/control/Tree.java 2005-09-09 11:10:57.000000000 +0200
@@ -1090,9 +1090,7 @@
html.append("</script>"); //$NON-NLS-1$
// contextmenu
- if (menu.getMenuItems().size() != 0) { html.append(menu.getHtml()); - }
// register menu
html.append("<script>" + this.getJavascriptTree() + ".menu = " + menu.getName() + "</script>"); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$