Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 507618 - CreateEditBasedElementCommand does not use the factory registered in the resource set
Summary: CreateEditBasedElementCommand does not use the factory registered in the reso...
Status: RESOLVED FIXED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Core (show other bugs)
Version: 2.0.1   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: 3.0.0   Edit
Assignee: Christian Damus CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 467545
  Show dependency tree
 
Reported: 2016-11-16 10:03 EST by Christian Damus CLA
Modified: 2017-05-10 04:21 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-16 10:03:58 EST 
Papyrus-based applications such as Papyrus-RT may sometimes need to install their own EFactory instances in the resource set's package registry to have a custom implementation of the UML metamodel instantiated with custom behaviour/API.  This means that all loci of element creation in Papyrus must use the registered package to create new UML model elements, not just invoke the UMLFactory.eINSTANCE static instance.

One such locus is the CreateEditBasedElementCommand, which currently uses the static factory to create new model elements.  This is used by, at least, the Model Explorer's "New Child" menu.

I set this as blocking the Papyrus-RT bug for the custom UML-RT implementation of the UML metamodel because that will be useless without fixing this element creation behaviour in Papyrus.
Comment 1 Eclipse Genie CLA 2016-11-16 10:39:26 EST 
New Gerrit change created: https://git.eclipse.org/r/85148
Comment 2 Eclipse Genie CLA 2016-11-16 11:28:36 EST 
Gerrit change https://git.eclipse.org/r/85148 was merged to [streams/2.0-maintenance].
Commit: http://git.eclipse.org/c/papyrus/org.eclipse.papyrus.git/commit/?id=cf3a9c9d3b1d9d2f73ec8fbd4762b804e5848ab3
Comment 3 Eclipse Genie CLA 2016-11-16 11:34:38 EST 
New Gerrit change created: https://git.eclipse.org/r/85158
Comment 5 Christian Damus CLA 2016-11-17 08:50:20 EST 
(In reply to Eclipse Genie from comment #2)
> Gerrit change https://git.eclipse.org/r/85148 was merged to
> [streams/2.0-maintenance].

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