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

Add syntax highlighting for restricted database keywords #6

Open
naxty opened this issue Apr 4, 2016 · 5 comments
Open

Add syntax highlighting for restricted database keywords #6

naxty opened this issue Apr 4, 2016 · 5 comments

Comments

@naxty
Copy link

naxty commented Apr 4, 2016

I added an attribute called 'from' to an entity and wanted to change my database from mongodb to mysql. My JUnit-Tests failed and I did not know why they failed. After some time I remembered I used 'from' as an attribute name. I renamed it and it work.
So I thought it would be nice with the jdl studio syntax shows some reversed keywords from database. What do you think?

@deepu105
Copy link
Member

deepu105 commented Apr 5, 2016

it would be good to have but we would have to maintain a hard coded list of keywords for this :( It would be very long list if we are to take all supported DB into consideration

@sendilkumarn
Copy link
Member

can we do something with this ? https://github.com/jgornick/reservedwordsearch

@deepu105
Copy link
Member

deepu105 commented Aug 1, 2016

That would be too much, may be we could move all reserved words to
jhipster-core and refer from there. I could write a simple plugin to lookup
that

Thanks & regards,
Deepu
On 1 Aug 2016 23:04, "Sendil Kumar N" [email protected] wrote:

can we do something with this ?
https://github.com/jgornick/reservedwordsearch


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#6 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABDlF_AtHKcfUUPMNL5ZPTRc0u_-r092ks5qbi4kgaJpZM4H-6QP
.

@MathieuAA
Copy link
Member

That's already done Deepu, check your inbox =)

@MathieuAA
Copy link
Member

Update: I'll release everything tomorrow... I have to check something first.

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

No branches or pull requests

4 participants