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

Bug 351172

Summary: Pop-up bar not working correctly in compartments that are placed inside of compartments
Product: [Modeling] GMF-Tooling Reporter: Ralph Gerbig <ralphgerbig>
Component: CoreAssignee: Project Inbox <gmp.gmf-tooling-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: borlander, mistria
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Registers child nodes to ModelingAssitantProvider none

Description Ralph Gerbig CLA 2011-07-05 08:03:54 EDT 
Build Identifier: M20100909-0800

Hi all,

I have the following hirarchy in my model:

diagram
  - compartment 1
   - compartment 2

I place model elements in compartment 2 that can be connected with each other. I would expect the popup bar to enable me to draw these connections. However, it allows me to only draw connections to notes.

Ralph


Reproducible: Always
Comment 1 Ralph Gerbig CLA 2011-07-05 08:22:28 EDT 
XXXModelingAssistantProvider does not seem to be called on those EditParts..

(In reply to comment #0)
> Build Identifier: M20100909-0800
> Hi all,
> I have the following hirarchy in my model:
> diagram
>   - compartment 1
>    - compartment 2
> I place model elements in compartment 2 that can be connected with each other.
> I would expect the popup bar to enable me to draw these connections. However,
> it allows me to only draw connections to notes.
> Ralph
> Reproducible: Always
Comment 2 Michael Golubev CLA 2011-07-25 10:07:14 EDT 
See the Bug #325683 for more details of the problem
Comment 3 Ralph Gerbig CLA 2011-08-02 07:58:35 EDT 
Created attachment 200705 [details]
Registers child nodes to ModelingAssitantProvider

The problem is in the original template. It registers only top level nodes to the ModelingAssistantProviders extension point. This modification also adds child nodes. Better coding style would be to make a union of child nodes and top level nodes instead of two foreach statements. However, I did not get this running.
Comment 4 Mickael Istria CLA 2011-10-28 09:17:55 EDT 
*** Bug 325683 has been marked as a duplicate of this bug. ***
Comment 5 Mickael Istria CLA 2011-10-28 10:02:21 EDT 
Patch tested, ok locally, committed, waiting for IC build.