Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 359863

Summary: Support for complex annotation attributes
Product: [Technology] Tigerstripe Reporter: Navid Mehregani <nmehrega>
Component: AnnotationsAssignee: Project Inbox <tigerstripe.annotations-inbox>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: marc.flauw, mshulgina, skanton
Version: unspecified   
Target Milestone: 0.5M0   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Attachments:
Description Flags
Detailed analysis
none
First pass on annotations' reference attributes.
none
Second pass on implementing this feature.
none
Second pass on implementing this feature. none

Description Navid Mehregani CLA 2011-10-04 12:05:04 EDT
Build Identifier: 

Certain annotation attributes need to reference other model components.  At the moment this is done by referencing the FQN/name of the model component as an EString type.  This is error prone and time consuming.  This enhancement will provide the ability to select specific model components in the UI.

Please see attached document for more details.

Reproducible: Always
Comment 1 Navid Mehregani CLA 2011-10-04 12:10:29 EDT
Created attachment 204522 [details]
Detailed analysis
Comment 2 Semyon Koptelov CLA 2011-10-24 20:01:47 EDT
Created attachment 205872 [details]
First pass on annotations' reference attributes.

Supports backwards-compatible GUI over strings as well as new ModelReference model object. GUI customizations should be done by configuring/overriding ModelReferenceCellEditor. See o.e.t.annotation.example plugin for examples.

Does not support lists yet.


Use these options to apply the patch:
* Apply the patch to the workspace root
* Ignore leading path name segments = 2
Comment 3 Semyon Koptelov CLA 2011-10-27 18:13:43 EDT
Created attachment 206100 [details]
Second pass on implementing this feature.

Supports lists.

Use these options to apply the patch:
* Apply the patch to the workspace root
* Ignore leading path name segments = 2
Comment 4 Semyon Koptelov CLA 2011-10-27 18:42:19 EDT
Created attachment 206102 [details]
Second pass on implementing this feature.

Second pass on implementing this feature. (removed irrelevant patch section)

Supports lists.

Use these options to apply the patch:
* Apply the patch to the workspace root
* Ignore leading path name segments = 2
Comment 5 Anton Salnik CLA 2011-10-31 07:17:52 EDT
http://wiki.eclipse.org/IModelComponent_References_in_Annotation_Attributes wiki page added.
Comment 6 Navid Mehregani CLA 2011-10-31 12:06:20 EDT
Code review was completed by Richard and myself on Oct 31 2011.
Comment 7 Anton Salnik CLA 2011-11-01 07:30:05 EDT
Patch applied.
Comment 8 Navid Mehregani CLA 2011-11-03 11:14:20 EDT
Verified. Thanks!
Comment 9 Navid Mehregani CLA 2012-03-01 16:00:33 EST
*** Bug 320068 has been marked as a duplicate of this bug. ***