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

SAXParseException in the language server trace when editing an .rng file #1441

Closed
datho7561 opened this issue Jan 9, 2023 · 1 comment · Fixed by #1449
Closed

SAXParseException in the language server trace when editing an .rng file #1441

datho7561 opened this issue Jan 9, 2023 · 1 comment · Fixed by #1449
Assignees
Labels
bug Something isn't working RelaxNG
Milestone

Comments

@datho7561
Copy link
Contributor

If you edit a RelaxNG file that uses the XML syntax, and the document becomes invalid, eg:

<grammar xmlns="http://relaxng.org/ns/structure/1.0"
  datatypeLibrary="http://www.w3.org/2001/XMLSchema-datatypes">
  <start>
    <ref name="root-element"/>
  </start>
  <define name="root-element">
      <element name="root-element">
        <ref name="asdf"></ref>
      </element>
  </define>
  <div>
    <div>
      <define name="asdf">
        <attribute name="child"></attribute>
        <element>
      </define>
    </div>
  </div>
</grammar>

(note the unclosed <element>)

Then a SAXParseException is output in the language server trace. It would be nice if this Exception didn't appear, since having an invalid XML file is expected when you are editing it.

@datho7561 datho7561 added bug Something isn't working RelaxNG labels Jan 9, 2023
@datho7561
Copy link
Contributor Author

I think I already filed this bug, so if someone finds that issue, please link it here and I'll close this one

angelozerr added a commit to angelozerr/lemminx that referenced this issue Jan 13, 2023
@datho7561 datho7561 added this to the 0.24.0 milestone Jan 13, 2023
@datho7561 datho7561 moved this to 👀 In review in IDE Cloudaptors Jan 13, 2023
datho7561 pushed a commit that referenced this issue Jan 13, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in IDE Cloudaptors Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working RelaxNG
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants