[MAGNOLIA-545] javascript error: 'mgnlTreeControlMenu is not defined' in tree views Created: 09/Sep/05  Updated: 23/Jan/13  Resolved: 17/May/06

Status: Closed
Project: Magnolia
Component/s: admininterface
Affects Version/s: 2.1 Final
Fix Version/s: 3.0 RC1

Type: Bug Priority: Major
Reporter: Michael Aemisegger Assignee: Boris Kraft
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: 5m
Time Spent: Not Specified
Original Estimate: 5m
Environment:

IE, Firefox


Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 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$



 Comments   
Comment by Philipp Bracher [ 17/May/06 ]

please reopen if not fixed in 3.0

Generated at Mon Feb 12 03:18:29 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.