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

Bug 476569

Summary: Custom Style Border nodes do not work with Edge creation tools
Product: [Modeling] Sirius Reporter: Maxime Porhel <maxime.porhel>
Component: DiagramAssignee: Project Inbox <sirius.diagram-inbox>
Status: RESOLVED INVALID QA Contact:
Severity: normal    
Priority: P3 CC: esteban.dugueperoux, maxime.porhel, pierre-charles.david, shaffdog78
Version: 2.0.5Keywords: triaged
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Bug Depends on: 476114, 476570, 476572    
Bug Blocks:    

Description Maxime Porhel CLA 2015-09-03 10:40:15 EDT
+++ This bug was initially created as a clone of Bug #476114 +++

We create a border node that has a custom style supplied by a custom editpart with our own shape.  We also create a editPartProvider that provides our shape.  
We have a creation tool that connects these border nodes together.

If we use Sirius defined shapes as our Style the Connection Tool works just fine.  If we, however, use our custom style border node the connection tool does not work.

Supplied are a version of the basic family model from the example that illustrates the problem.  I define a basic custom style that is a simple square shape and try to use the connection tool with it and it fails to recognize that it is a valid connection point.  

The BasicFamilyModelWithCustomEditPart.zip contains the model code and the custom edit part.

The ModedAndDesign contains the model and the odesign file.

Import projects from the BasicFamilyModelWithCustomEditPart.zip and run a eclipse application.

Import the ModelAndDesign.zip into the running eclipse.

open a diagram and try to connection the border nodes to each other with the custom style active. It will not allow it.

modify the odesign file and set the ConditionalStyle expression to self = null to allow the sirius defined shape style to be selected and try the connection again.  It should be allowed this time.

Thanks,
Comment 1 Pierre-Charles David CLA 2015-09-16 03:52:25 EDT
Unneeded clone: was already fixed on all active branches.