Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 506896 - F2 rename action does not use edit helpers
Summary: F2 rename action does not use edit helpers
Status: RESOLVED FIXED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Core (show other bugs)
Version: 2.1.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.0.0   Edit
Assignee: Christian Damus CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 502385
  Show dependency tree
 
Reported: 2016-11-02 09:15 EDT by Christian Damus CLA
Modified: 2017-05-10 04:17 EDT (History)
0 users

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 Christian Damus CLA 2016-11-02 09:15:19 EDT 
+++ This bug was initially created as a clone of Bug #502385 +++

In Papyrus-RT, if you rename a protocol using F2, then this only renames the actual collaboration of the protocol. The other internal elements, i.e. the <<ProtocolContainer>> Package and the three <<RTMessageSet>> Interfaces does not get renamed according to the naming rules for the internal elements of a protocol.

The root cause is that the rename action handler does not use the element-type edit-helper framework to obtain the command that sets the name, therefore by-passing the advice commands in Papyrus-RT and any other DSML.
Comment 1 Eclipse Genie CLA 2016-11-02 10:16:32 EDT 
New Gerrit change created: https://git.eclipse.org/r/84345
Comment 2 Eclipse Genie CLA 2016-11-02 11:37:34 EDT 
New Gerrit change created: https://git.eclipse.org/r/84355
Comment 3 Eclipse Genie CLA 2016-11-02 11:59:49 EDT 
Gerrit change https://git.eclipse.org/r/84345 was merged to [streams/2.0-maintenance].
Commit: http://git.eclipse.org/c/papyrus/org.eclipse.papyrus.git/commit/?id=4a1cb9f9a8f065c323065bad3fbdc0027f2543b2
Comment 5 Christian Damus CLA 2016-11-03 11:30:55 EDT 
(In reply to Eclipse Genie from comment #3)
> Gerrit change https://git.eclipse.org/r/84345 was merged to
> [streams/2.0-maintenance].

(In reply to Eclipse Genie from comment #4)
> Gerrit change https://git.eclipse.org/r/84355 was merged to [master].