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

Bug 498258

Summary: The label of the property view does not refresh on a null value
Product: [Modeling] Sirius Reporter: Arthur Daussy <arthur.daussy_ssii>
Component: PropertiesAssignee: Stephane Begaudeau <stephane.begaudeau>
Status: CLOSED FIXED QA Contact: Julien Dupont <julien.dupont>
Severity: normal    
Priority: P3 CC: julien.dupont, maxime.porhel, pierre-charles.david, steve.monnier
Version: 4.0.0Keywords: triaged
Target Milestone: 4.1.0   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/81063
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=591a9074d3cc3d396a5c272ac29ca4c38c5c0803
Whiteboard:
Attachments:
Description Flags
Use case to reproduce problem none

Description Arthur Daussy CLA 2016-07-21 05:00:10 EDT 
I'm using Sirius with the new property framework. The label on the top of the property view is not properly refreshed when a user select an element that has a label equal to null. Instead it keeps the label of the last selected item.

To reproduce:
* Open your Sirius Editor with some property views
* Select element that has a label (The itemProvider of this element returns a getText() != null)
* The select an element with no label The itemProvider of this element returns a getText() == null)

The label of the property view keeps the label of first selected items
Comment 1 Steve Monnier CLA 2016-07-21 08:23:46 EDT 
Hi,
thanks for your detailled bug report.
Comment 2 Eclipse Genie CLA 2016-09-14 04:15:33 EDT 
New Gerrit change created: https://git.eclipse.org/r/81063
Comment 4 Arthur Daussy CLA 2016-09-21 10:31:18 EDT 
It's corrected on 
* Sirius 4.1.0.201609201603
* Eef:   1.7.0.201609191510

Thanks
Comment 5 Julien Dupont CLA 2016-10-04 08:23:03 EDT 
Created attachment 264570 [details]
Use case to reproduce problem

Steps to reproduce:
- Open the diagram named "New Diagram"
- Select the EClass named "E1", the properties view should display EClass Name: E1
- OK
- Select the EClass not named, the properties view should be updated.
Comment 6 Pierre-Charles David CLA 2016-10-18 11:07:31 EDT 
Available in Sirius 4.1.0, see https://wiki.eclipse.org/Sirius/4.1.0 for details.