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

Aggregate errors in xsd:import|include@schemaLocation for referenced grammar which have errors #1117

Merged
merged 1 commit into from
Oct 6, 2021

Conversation

angelozerr
Copy link
Contributor

@angelozerr angelozerr commented Sep 30, 2021

Aggregate errors in xsd:import|include@schemaLocation for referenced grammar which have errors

Fixes redhat-developer/vscode-xml#596

Signed-off-by: azerr azerr@redhat.com

@angelozerr angelozerr force-pushed the xsd-aggregate-errors branch 2 times, most recently from aaa0ac8 to e3942b2 Compare September 30, 2021 09:12
@angelozerr
Copy link
Contributor Author

angelozerr commented Sep 30, 2021

This PR fixes

image

  • when you use xsd:import and xsd:include with bad schemaLocation attribute value, now the error should highlight the schemaLocation attribute value

image

  • check some NPE that I discover when tests are executed (because in some test SharedSettings and languageclient can be null).

The PR can be tested, I need to write tests and after that I will swith to real PR.

@angelozerr angelozerr changed the title Confusing error messages when XML schema location redirects to an HTML page Aggregate errors in xsd:import|include@schemaLocation for referenced grammar which have errors Sep 30, 2021
@angelozerr angelozerr force-pushed the xsd-aggregate-errors branch 2 times, most recently from ce5fc6f to 30cf4a2 Compare October 5, 2021 09:19
@angelozerr angelozerr marked this pull request as ready for review October 5, 2021 09:20
grammar which have errors

Fixes redhat-developer/vscode-xml#596

Signed-off-by: azerr <azerr@redhat.com>
@angelozerr
Copy link
Contributor Author

@rgrunber I have written test, you can review it now.

@rgrunber
Copy link
Contributor

rgrunber commented Oct 5, 2021

The change works for me, and I don't see any errors that are related so I would feel free to merge.

@angelozerr
Copy link
Contributor Author

@rgrunber thanks for the review!

@angelozerr angelozerr merged commit f70a0f8 into eclipse:master Oct 6, 2021
@angelozerr angelozerr added this to the 0.18.1 milestone Oct 6, 2021
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

Successfully merging this pull request may close these issues.

Aggregate errors in xsd:import|include@schemaLocation for referenced grammar which have errors
3 participants