516895 – In VSM editor, "extends" element label is not updated when the value is changed

Bug 516895 - In VSM editor, "extends" element label is not updated when the value is changed

Summary: In VSM editor, "extends" element label is not updated when the value is changed

Status:	CLOSED FIXED

Alias:	None

Product:	Sirius
Classification:	Modeling
Component:	Properties (show other bugs)
Version:	5.0.0
Hardware:	All All

Importance:	P3 normal (vote)
Target Milestone:	5.1.0
Assignee:	Pierre-Charles David
QA Contact:

URL:
Whiteboard:	backport vsm need_test
Keywords:	triaged

Depends on:
Blocks:

Reported:	2017-05-18 10:33 EDT by Jessy Mallet
Modified:	2017-11-08 03:37 EST (History)
CC List:	2 users (show)

See Also:	Gerrit Change Git Commit

Attachments
Project to import in order to reproduce the bug (18.97 KB, application/x-zip-compressed) 2017-05-18 10:33 EDT, Jessy Mallet	no flags	Details
ScreenShot to show the unsynchronised label. (32.20 KB, image/png) 2017-05-18 10:34 EDT, Jessy Mallet	no flags	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Jessy Mallet

2017-05-18 10:33:27 EDT

Created attachment 268445 [details]
Project to import in order to reproduce the bug

When you extend a widget (or a group or a page), the label on the VSM editor is not update with the new value in "extends" Field. You have to close the editor and re-open it to have the good one.

Step to reproduce :
* import the attached project,
* open the basicfamily.odesign,
* open "Person Group" in properties views,
* change "Extends" field by using "name2" widget and save,
* the labet for the extend element is not synchronised : KO

Comment 1 Jessy Mallet

2017-05-18 10:34:01 EDT

Created attachment 268446 [details]
ScreenShot to show the unsynchronised label.

Comment 2 Pierre-Charles David

2017-06-15 12:16:08 EDT

Confirmed, thanks for finding this.

Comment 3 Eclipse Genie

2017-08-28 11:43:39 EDT

New Gerrit change created: https://git.eclipse.org/r/103781

Comment 4 Eclipse Genie

2017-08-30 08:08:53 EDT

Gerrit change https://git.eclipse.org/r/103781 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=fb30021312b3aa6bec4d122d9c222b662450e107

Comment 5 Pierre-Charles David

2017-08-30 08:10:19 EDT

Fixed by fb30021312b3aa6bec4d122d9c222b662450e107. Still needs an automated test.

Comment 6 Pierre-Charles David

2017-08-31 10:57:39 EDT

Marking as FIXED for 5.1.0. The properties views plug-ins are not yet present in the context of our test suites, so this can't be tested automatically until then (and it's a non-trivial task).

Comment 7 Pierre-Charles David

2017-11-08 03:37:14 EST

Available in Sirius 5.1.0, see https://wiki.eclipse.org/Sirius/5.1.0.