Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364919 - Non-containment one-to-many references to EMap entries not generated to EList
Summary: Non-containment one-to-many references to EMap entries not generated to EList
Status: CLOSED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Ed Merks CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 384591 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-11-27 15:22 EST by Nikolay Kasyanov CLA
Modified: 2012-07-09 06:57 EDT (History)
1 user (show)

See Also:

Attachments
test model with generated code (15.67 KB, application/zip)
2011-11-27 15:25 EST, Nikolay Kasyanov CLA 		no flags Details
View All 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 Nikolay Kasyanov CLA 2011-11-27 15:22:42 EST 
Build Identifier: 20110916-0149

Non-containment one-to-many references to Map Entry classes (i.e. class with key & value features and instance class java.util.Map$Entry) not generated to EList

Initial forum post: http://www.eclipse.org/forums/index.php?t=rview&goto=759224#msg_759224

Reproducible: Always

Steps to Reproduce:
1. Create an Entry class (i.e. with key & value features and instance class java.util.Map$Entry).
2. Create non-containment one-to-many reference "refToEntry" to Entry in some EClass, say DomainClass.
3. Generate code.
4. See DomainClassImpl, type of "refToEntry" field is Entry<K, V> instead of EList<Entry<K,V>>.
Comment 1 Nikolay Kasyanov CLA 2011-11-27 15:25:57 EST 
Created attachment 207585 [details]
test model with generated code

Model with possible test case for non-containment references to map entries.

Problem can be found in LanguageImpl class, line 47
Comment 2 Ed Merks CLA 2011-11-28 03:16:51 EST 
A fix is committed to CVS for 2.8.
Comment 3 Nikolay Kasyanov CLA 2011-11-28 05:39:08 EST 
Cool!
Is it template only fix so it can be easily used with current EMF?
EMF CVS is here, am I right?
:pserver:anonymous@dev.eclipse.org:/cvsroot/modeling
Comment 4 Ed Merks CLA 2011-11-28 05:41:03 EST 
No, all the changes are in GenTypedElementImpl.

Yes, EMF is under there in CVS.
Comment 5 Nikolay Kasyanov CLA 2011-12-02 17:03:14 EST 
Thanks!
Another question, will it be in Indigo SR2 or only in Juno?
Comment 6 Ed Merks CLA 2011-12-02 17:18:22 EST 
It's only destined for Juno.
Comment 7 Ed Merks CLA 2012-01-09 04:00:18 EST 
The fix is available in a build.
Comment 8 Ed Merks CLA 2012-07-09 06:57:27 EDT 
*** Bug 384591 has been marked as a duplicate of this bug. ***