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

Bug 501903

Summary: The 'selection' variable on hyperlink widget carry an incorrect value (self)
Product: [Modeling] Sirius Reporter: Arthur Daussy <arthur.daussy_ssii>
Component: PropertiesAssignee: Melanie Bats <melanie.bats>
Status: CLOSED FIXED QA Contact: Julien Dupont <julien.dupont>
Severity: normal    
Priority: P3 CC: julien.dupont, maxime.porhel, pierre-charles.david
Version: 4.0.0Keywords: triaged
Target Milestone: 4.1.2   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/81681
https://git.eclipse.org/c/eef/org.eclipse.eef.git/commit/?id=4e21ca9e6ddd5fa3b0d9b72ebc09a720bdde5e93
https://git.eclipse.org/r/84358
https://git.eclipse.org/c/eef/org.eclipse.eef.git/commit/?id=0dcc9b430de1602d054611634947cd0b981fafc5
Whiteboard:
Attachments:
Description Flags
Use case to reproduce problem
none
This new use case worsk fine. The first is faulty none

Description Arthur Daussy CLA 2016-09-21 06:02:06 EDT 
Working on version: 
* Sirius 4.1.0.201609201603
* Eef:   1.7.0.201609191510

In the previous version we used ReferenceWidget that had a variable 'selection' that carried the value of the widget (in our case the value of a mono referenced feature). In the new version, the odesign has transformed the widget reference into an hyperlink and that's great. However the selection variable that used to carry the value of the reference now carry the current selected EObject.
For us that a problems since we can not plug our "navigation" expression on this widget.

Regards.
Comment 1 Maxime Porhel CLA 2016-09-22 03:34:10 EDT 
Hi Arthur, 

Thanks for the bug report.
Comment 2 Eclipse Genie CLA 2016-09-22 08:08:32 EDT 
New Gerrit change created: https://git.eclipse.org/r/81681
Comment 4 Julien Dupont CLA 2016-10-04 09:53:10 EDT 
Created attachment 264571 [details]
Use case to reproduce problem

Steps to reproduce:
- Open Diagram "New Diagram"
- Select Attribute E1 (should have an attribute named "a1")
- In properties view Tab Page
   -- Click on link "a1"
A new representation should be opened and the element in, should have name "a1Renamed".
Comment 5 Pierre-Charles David CLA 2016-10-18 11:08:46 EDT 
Available in Sirius 4.1.0, see https://wiki.eclipse.org/Sirius/4.1.0 for details.
Comment 6 Eclipse Genie CLA 2016-11-02 12:24:34 EDT 
New Gerrit change created: https://git.eclipse.org/r/84358
Comment 8 Julien Dupont CLA 2016-12-01 10:58:39 EST 
Created attachment 265681 [details]
This new use case worsk fine. The first is faulty
Comment 9 Pierre-Charles David CLA 2016-12-08 11:12:57 EST 
Available in Sirius 4.1.2 (see https://wiki.eclipse.org/Sirius/4.1.2 for details).