Bug 364508

Description Alexander Nittka 2011-11-22 14:47:18 EST

In the first iteration this is a question rather than a bug report. Take the domainmodel example and override the ValidID - adding a keyword to be allowed in a valid Id - rule to:

ValidID:ID|"op";

Name the package in a sample model file op.mypackage

The result is that the name (i.e. the string stored in the name feature) of the package is indeed only "mypackage" (see outline view and index etc.) The reason is that the QualifiedNameValueConverter in its toValue-method iterates only over the *leaf nodes*. So although the value converter for ValidID might already be correct or is overridden to be correct, it is not used at all in the QualifiedNameValueConverter.
The leafs will be ID and Keyword, but a keyword cannot even be processed by a delegate rule call, so the QualifiedNameValueConverter would have to be "reimplemented".

Question: is this intended or is it a bug? My impression from the grammar design is that the point of the ValidID rule is being able to change what an ID may look like without touching the QualifiedNameRule (and hence not touch its value converter) at all. I would understand having to modify the QualifiedNameValueConverter if I actually change the QualifiedName rule...