Fix for Bug 401709 - Secondary Problems view doesn't have a view menu ->

not configurable (menus / TBs)

1 files changed, 15 insertions, 2 deletions

diff --git a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/e4/compatibility/CompatibilityView.java b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/e4/compatibility/CompatibilityView.java
index 498795a1eff..a155504063e 100644
--- a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/e4/compatibility/CompatibilityView.java
+++ b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/e4/compatibility/CompatibilityView.java
@@ -127,7 +127,13 @@ public class CompatibilityView extends CompatibilityPart {
 		MMenu menu = getViewMenu();
 		if (menu == null) {
 			menu = MenuFactoryImpl.eINSTANCE.createMenu();
-			menu.setElementId(part.getElementId());
+
+			// If the id contains a ':' use the part before it as the descriptor
+			// id
+			String partId = part.getElementId();
+			int colonIndex = partId.indexOf(':');
+			String descId = colonIndex == -1 ? partId : partId.substring(0, colonIndex);
+			menu.setElementId(descId);
 
 			menu.getTags().add(StackRenderer.TAG_VIEW_MENU);
 			menu.getTags().add(ContributionsAnalyzer.MC_MENU);
@@ -144,7 +150,14 @@ public class CompatibilityView extends CompatibilityPart {
 		MToolBar toolbar = part.getToolbar();
 		if (toolbar == null) {
 			toolbar = MenuFactoryImpl.eINSTANCE.createToolBar();
-			toolbar.setElementId(part.getElementId());
+
+			// If the id contains a ':' use the part before it as the descriptor
+			// id
+			String partId = part.getElementId();
+			int colonIndex = partId.indexOf(':');
+			String descId = colonIndex == -1 ? partId : partId.substring(0, colonIndex);
+			toolbar.setElementId(descId);
+
 			part.setToolbar(toolbar);
 		}
 		apr = rendererFactory.getRenderer(toolbar, parent);