Community
Participate
Working Groups
+++ This bug was initially created as a clone of Bug #435808 +++ discovered during Sirius 1.0.0 RC2 homologation on Eclipse Luna SDK RC2 See Bug 435808 for more details. The popupAny contributions (locationUri popup:org.eclipse.ui.popup.anybefore=additions for example) are added twice in org.eclipse.e4.ui.internal.workbench.ContributionsAnalyzer.XXXgatherMenuContributions(MMenu, List<MMenuContribution>, String, ArrayList<MMenuContribution>, ExpressionContext, boolean) : if (!filtered && menuContribution.isToBeRendered() && popupAny) { // process POPUP_ANY first toContribute.add(menuContribution); } else if (filtered || (!popupTarget && !parentID.equals(id)) || !menuContribution.isToBeRendered()) { continue; } includedPopups.add(menuContribution); } toContribute.addAll(includedPopups); The if/else if block should be improved to avoid duplicated entries or a LinkedHAshSet should be used.
See https://git.eclipse.org/r/27392 for a proposed correction. It corrects the duplication in Sirius (Bug 435808).
According to bug 435808 comment 1 this is a regression compared to 4.4 M7, hence something we have to look at for 4.4.
Daniel, could you please review this patch? PW
+1 component lead PW
*** Bug 436018 has been marked as a duplicate of this bug. ***
Thanks Maxime. released as http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=c79fc3a4f91cc14e8bf52c02aed6c1d1891e2912 PW
*** Bug 435414 has been marked as a duplicate of this bug. ***
Verified in I20140528-2000.
*** Bug 436720 has been marked as a duplicate of this bug. ***