Bug 544515 - [Contributions] MenuContribution value of allPopups is ignored
Summary: [Contributions] MenuContribution value of allPopups is ignored
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.10   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords:
Depends on:
Blocks:
 
Reported: 2019-02-17 10:32 EST by Rolf Theunissen CLA
Modified: 2021-02-10 01:33 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rolf Theunissen CLA 2019-02-17 10:32:12 EST
Currently, the "allPopups" value in menuContribution extentions is ignored. By default, this value should have been "false", however the current behavior is "true" for all cases.

Probably it is a bad idea, to start using the "false" default again, so best it is to switch to a "true" default, or remove this value at all.

MenuPopulationTest.testPrivatePopup test the usage of the "allPopups" value, this test currently fails. Note that the test assumes "true" as default value.

From the extension definition:
By default popup contributions are not contributed to context menus that do not include an "additions" marker.
Comment 1 Lars Vogel CLA 2019-02-20 06:23:34 EST
I suggest to remove it, or do you see an issue with that Rolf?
Comment 2 Rolf Theunissen CLA 2019-02-20 08:12:33 EST
(In reply to Lars Vogel from comment #1)
> I suggest to remove it, or do you see an issue with that Rolf?

Effectively it is not used, for a long time now, and apparently nobody is complaining. So I don't see any problem in removing it.

Though it could result in cluttering popup menus that should not have been extended, or are extended at unpredictable locations.
Then again, it might be that the current framework always inserts an "additions" markers in menus (at least in some cases it does if I remember correctly).

Just wondering now what the behavior would be for:
<menuContribution
            locationURI="popup:org.eclipse.ui.popup.any?after=additions">
Without having a "additions" marker in a menu.
Comment 3 Dani Megert CLA 2019-02-20 08:35:37 EST
For 4.11 this is too late/risky.
Comment 4 Eclipse Genie CLA 2021-02-10 01:33:01 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.