460014 – Validate the static image paths mentioned in VSMs

Bug 460014 - Validate the static image paths mentioned in VSMs

Summary: Validate the static image paths mentioned in VSMs

Status:	CLOSED FIXED

Alias:	None

Product:	Sirius
Classification:	Modeling
Component:	Core (show other bugs)
Version:	1.0.0M6
Hardware:	All All

Importance:	P3 normal (vote)
Target Milestone:	3.0.0M6
Assignee:	Benjamin Grouhan
QA Contact:	Belqassim Djafer

URL:
Whiteboard:
Keywords:	triaged

Depends on:
Blocks:

Reported:	2015-02-16 08:26 EST by Pierre-Charles David
Modified:	2015-06-24 11:13 EDT (History)
CC List:	2 users (show)

See Also:	Gerrit Change Gerrit Change Git Commit Git Commit Gerrit Change Git Commit

Attachments
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 Pierre-Charles David

2015-02-16 08:26:28 EST

There are several places in a VSM where a specifier can point to an image file. When this is done in a static way (i.e. not an interpreter expression), during VSM validation we should check the paths are actually correct:
* they point to an existing and accessible file;
* they point to an image in one of formats supported by Sirius.

Comment 1 Eclipse Genie

2015-02-16 09:52:56 EST

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

Comment 2 Eclipse Genie

2015-02-18 05:57:35 EST

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

Comment 3 Eclipse Genie

2015-02-18 05:57:37 EST

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

Comment 4 Pierre-Charles David

2015-02-18 05:58:17 EST

Fixed by c2b81eeebf06dfdbf7e0ee21c02832788edec400 and d702ae161d41df2b957e73135d3831d8e2b69c03.

Comment 5 Eclipse Genie

2015-02-18 10:05:01 EST

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

Comment 6 Eclipse Genie

2015-02-18 10:29:11 EST

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

Comment 7 Belqassim Djafer

2015-04-13 11:17:15 EDT

Verified on Sirius 3.0.0M6

Comment 8 Belqassim Djafer

2015-05-22 03:43:29 EDT

Verified with Sirius 3.0.0 RC1

Comment 9 Pierre-Charles David

2015-06-24 11:13:38 EDT

Available in Sirius 3.0.0. See https://wiki.eclipse.org/Sirius/3.0.0.