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

JSON Schema support #45

Open
angelozerr opened this issue Mar 23, 2015 · 6 comments
Open

JSON Schema support #45

angelozerr opened this issue Mar 23, 2015 · 6 comments
Labels

Comments

@angelozerr
Copy link
Contributor

@ralfstx I'm developping a JSON Editor based on WTP https://github.com/angelozerr/eclipse-wtp-json (I will contribute to WTP soon).

I'm supporting JSON Schema for :

  • validation
  • completion.

by using JSON Schema at http://schemastore.org/

I have developed my own JSON Schema (works only for completion) because every JSON Schema written in Java doesn't support line number where there is an error. It's because the JSON source to validate doesn't contains line number, offset for each JsonValue.

I create this issue because I would liek to know if minimal-json could :

  • give line number, offset for each JsonValue (by default none line info, this feature will be avaiable with an option)
  • support a factory kind in teh JsonParser to create a JsonSchema instance instead of JsonObject instance.

If you are not aware to do that, close this issue and I will create my own minimal-json with those extension.

@ralfstx
Copy link
Owner

ralfstx commented Mar 24, 2015

Hi Angelo,
I see this as a part of #17. I plan on a streaming API that would allow you to hook into the parsing process and get the char positions for every element. This should also allow to create an alternative output such as JsonSchema.

@angelozerr
Copy link
Contributor Author

Thanks @ralfstx for your answer. For the moment I have copied/pasted minimal-json inside my project in order to update for my need by waiting for #17

If I can manage JsonSchema, are you interested to include it to minimal-json?

@angelozerr
Copy link
Contributor Author

@ralfstx for your information Eclipse WTP JSON is now included in the WTP git repository (it will available for Neon)!

So no I think the JsonSchema feature is a very important thing. When do you think could you have time to support #17 in order to get the char positions for every element.?

@mickaelistria @dgolovin @vrubezhny @maxandersen @alexeykazakov I think you should follow this issue for WTP JSON.

Many thanks @ralfstx for your help.

@ralfstx
Copy link
Owner

ralfstx commented Jan 27, 2016

@angelozerr I know I promised the streaming API for last year already. I still have it on my list for the next release and hope to get to it soon.

@angelozerr
Copy link
Contributor Author

Thanks @ralfstx for your answer. Is there any chance that you will implement Streaming API? If yes, when you will have time to do that?

@ralfstx
Copy link
Owner

ralfstx commented Mar 1, 2016

@angelozerr Ok, I guess it's time to make a commitment. I have been working on a draft lately and I'm going to reserve the weekend March 12/13 for finishing it. Thanks for your patience.

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

No branches or pull requests

2 participants