Bug 467711 - [Model Explorer] Double entries of the "Delete" menu choice on the context menu in Model Explorer
Summary: [Model Explorer] Double entries of the "Delete" menu choice on the context me...
Status: NEW
Alias: None
Product: Papyrus
Classification: Modeling
Component: Views (show other bugs)
Version: 1.1.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-20 09:26 EDT by Peter Cigehn CLA
Modified: 2017-09-08 09:54 EDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Cigehn CLA 2015-05-20 09:26:40 EDT 
There is a duplication of the "Delete" menu choice on the context menu in the Model Explorer, one at the top of the context with the name "Delete..." and one more further down, above the "Rename" menu choice, named "Delete".

Steps to reproduce:

1) Make a clean install of Mars M7 modeling package
2) Install only the Papyrus UML from main (no extra to avoid the possibility of some extra component contributing the duplicate menu choice).
3) Create a new Papyrus UML model
4) Add one model element below the root element
5) Right click on the added model element to bring up the context menu

Result: One "Delete..." menu choice at top and one duplicated "Delete" above the "Rename" menu choice.
Comment 1 Camille Letavernier CLA 2015-05-21 05:28:32 EDT 
The "Delete" at top is suspicious, but I can't reproduce it

Which version of Eclipse did you install? Standard/Classic/Committer, or Modeling?
Comment 2 Patrik Nandorf CLA 2015-05-21 06:08:13 EDT 
I did some test and what I found was that both menu items go to the same handler from what I could see.
Comment 3 Peter Cigehn CLA 2015-05-21 06:36:58 EDT 
(In reply to Camille Letavernier from comment #1)
> The "Delete" at top is suspicious, but I can't reproduce it
> 
> Which version of Eclipse did you install? Standard/Classic/Committer, or
> Modeling?

I made a clean install starting with the Eclipse Modeling Tools M7 package:

https://www.eclipse.org/downloads/download.php?file=/technology/epp/downloads/release/mars/M7/eclipse-modeling-mars-M7-win32.zip

Then I installed the latest Mars build of Papyrus UML only (no extra components).
Comment 4 Camille Letavernier CLA 2015-05-21 06:51:00 EDT 
> I made a clean install starting with the Eclipse Modeling Tools M7 package:

I suppose the chances are high that a Modeling Component installs a delete menu on everything that looks like an EObject (Although that would certainly be a bad idea)

I'll install the modeling tools and see what happens

> I did some test and what I found was that both menu items go to the same handler from what I could see.

Papyrus uses the standard Eclipse Delete command, with its own handler. So, I'm not surprised that the behavior of both menu entries is the same, since they probably both use the same standard delete command.
Comment 5 Camille Letavernier CLA 2015-05-21 09:00:57 EDT 
Confirmed on Modeling M7 + Papyrus

This menu is not contributed by Papyrus itself