Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Information missing for Attributes change operators #25

Open
Jefidev opened this issue Nov 14, 2019 · 2 comments
Open

Information missing for Attributes change operators #25

Jefidev opened this issue Nov 14, 2019 · 2 comments
Assignees

Comments

@Jefidev
Copy link

Jefidev commented Nov 14, 2019

For the query evolution module, I am relying on the changeOperator syntax.

I started to work on the Attribute evolution and the information about the entity containing the attribute to evolve is missing.

I suggest to change this kind of syntax :
rename' 'attribute' attributeToRename=[Attribute|EString] 'as' newName=EString

By this :
rename' 'attribute' attributeToRename=[Attribute|EString] 'from' entity=[Entity|EString] 'as' newName=EString

or something equivalent.

@jdirocco
Copy link
Member

jdirocco commented Nov 14, 2019

Dear @Jefidev ,
RenameAttribute change operator doesn't contain the entity relation because it is redundant.
attributeToRename is contained in a Entity and you can access it by eContainmanet() operation/method. Why should we introduce this new field?

@Jefidev
Copy link
Author

Jefidev commented Nov 14, 2019

Hello,
I am working on a feature enabling user of TyphonQL to evolve their queries based on change operators like this :

changeOperators [ 
	rename entity Product as Item 
	rename attribute prod as product_id
]

// Insert queries here
from Product p select p where p.id == "1230",
from Product p, Stock s select p, s where p.id == "30",
from Person p select p.name,

The changeOperators could be copy-pasted from a .tml file by the programmer or written by him directly. Thus, this script doesn't rely especially on an .xmi.

If the change operators completely describe the changes this is not an issue but some information are missing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants