Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 508404 - [Copy/Paste] Copy and paste does not use registered EFactory
Summary: [Copy/Paste] Copy and paste does not use registered EFactory
Status: RESOLVED FIXED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Core (show other bugs)
Version: 2.0.1   Edit
Hardware: PC All
: P2 major (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-29 12:32 EST by Christian Damus CLA
Modified: 2017-05-10 04:19 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-29 12:32:23 EST 
Papyrus-RT installs (or, rather, will have to install) a custom implementation of the UML Metamodel for UML-RT semantics (this is bug 467545).  In order for this to work, all mechanisms in Papyrus that load UML resources and that create UML model elements must use the factory installed in the ResourceSet to create new objects.

One such locus of object creation (or, rather at least four loci) is in the Copy/Paste infrastructure.  This uses EcoreUtil.Copier to copy model elements, and this copier framework by default uses the statically registered EFactory instances.  We need this to be customized to use the resource set's registered factory, instead.
Comment 1 Eclipse Genie CLA 2016-11-29 12:51:20 EST 
New Gerrit change created: https://git.eclipse.org/r/85963
Comment 2 Eclipse Genie CLA 2016-11-29 13:38:59 EST 
New Gerrit change created: https://git.eclipse.org/r/85969
Comment 4 Eclipse Genie CLA 2016-11-29 16:21:15 EST 
Gerrit change https://git.eclipse.org/r/85963 was merged to [streams/2.0-maintenance].
Commit: http://git.eclipse.org/c/papyrus/org.eclipse.papyrus.git/commit/?id=6e9931ef995ebed98ea22fd825f4ac5db7cccf16
Comment 5 Christian Damus CLA 2016-11-29 16:23:22 EST 
(In reply to Eclipse Genie from comment #4)
> Gerrit change https://git.eclipse.org/r/85963 was merged to
> [streams/2.0-maintenance].

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