507065 – Add support for package::Class syntax for domain classes

Bug 507065 - Add support for package::Class syntax for domain classes

Summary: Add support for package::Class syntax for domain classes

Status:	CLOSED FIXED

Alias:	None

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

Importance:	P3 enhancement (vote)
Target Milestone:	5.0.0M6
Assignee:	Pierre-Charles David
QA Contact:

URL:
Whiteboard:
Keywords:	noteworthy, triaged

Duplicates (1):	465947 (view as bug list)
Depends on:
Blocks:

Reported:	2016-11-04 12:07 EDT by Pierre-Charles David
Modified:	2017-06-29 03:31 EDT (History)
CC List:	3 users (show)

See Also:	Gerrit Change Git Commit Gerrit Change Gerrit Change Git Commit Gerrit Change 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

2016-11-04 12:07:18 EDT

In the VSM, the "domainClass" attributes only support a syntax of the form "package.Class", which is not consistent with the syntax used by AQL, the query language we recommend. We should allow users to use the "package::Class" syntax and interpret it as equivalent.

Comment 1 Cedric Brun

2016-11-04 13:07:47 EDT

*** Bug 465947 has been marked as a duplicate of this bug. ***

Comment 2 Eclipse Genie

2017-02-03 03:54:56 EST

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

Comment 3 Pierre-Charles David

2017-02-03 04:01:01 EST

Commit cdc0960f3b9107a1138fd3f1a527a4e73eec255e adds the support for the new syntax. Before closing this ticket we'll need to update the documentation:
* mention the new feature in the release notes;
* change the actual doc content to mention the new syntax. Maybe even make it the new standard/recommended syntax, the old "p.C" becoming legacy.
* update the screenshots where domainClass appears.

Comment 4 Eclipse Genie

2017-03-07 11:27:18 EST

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

Comment 5 Eclipse Genie

2017-03-07 12:09:06 EST

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

Comment 6 Eclipse Genie

2017-03-09 09:04:45 EST

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

Comment 7 Pierre-Charles David

2017-03-09 09:05:41 EST

Fixed. The new syntax is now the default in the documentation and in completion proposals.

Comment 8 Eclipse Genie

2017-03-17 05:04:51 EDT

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

Comment 9 Eclipse Genie

2017-03-20 08:48:00 EDT

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

Comment 10 Jessy Mallet

2017-05-15 12:09:27 EDT

Validated with Sirius 5.0.0.201705121003

Comment 11 Pierre-Charles David

2017-05-16 03:47:54 EDT

Verifier by Jessy.

Comment 12 Eclipse Genie

2017-06-08 09:16:10 EDT

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

Comment 13 Eclipse Genie

2017-06-12 10:08:23 EDT

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

Comment 14 Pierre-Charles David

2017-06-29 03:31:34 EDT

Available in Sirius 5.0.0, see https://wiki.eclipse.org/Sirius/5.0.0 for details.