OpenQuoteExpected error for ATTLIST breaks DTD validation

[ejgutierrez74]

Im trying to validate xml file against DTD file.
The problem is that vscode-xml only says that is an error in DTD file...no more info...and then is difficult to debug...

But if you use a internet/online validator the info is much clearer ( for example https://www.xmlvalidation.com/ ):

or this:

So im missing something ? is there any chance that vscode-xml should show a list of errors/warnings.like when you test if an xml file is well formed ? Would be really useful...

[angelozerr]

Could you share your xml and ďtd files please.

[ejgutierrez74]

Yes of course
For example these ones....many errors to debug...

bautistamonicaoriginal.zip

Also once corrected:

monicacorrected.zip

[ejgutierrez74]

Also you can make something better with netbeans....
https://www.youtube.com/watch?v=DQpX9bns-d0

But id like to see all the errors at once generated, as vscode-xml do it when xml file is not well formed.
Seems that netbeans only shows the first error, at least is something better than this extension...

The better place i found is the online validator linked in previous post...

Thanks

[ejgutierrez74]

Edited first post, because i uploaded the same image twice...No you can see at least two errors validating the xml against a DTD file...

[fbricon]

Testing your sample files, I found an error in the XML language server:

[Error - 14:12:17] Nov 13, 2023 02:12:17 org.eclipse.lemminx.extensions.dtd.participants.diagnostics.DTDValidator doDiagnostics()
Message: Unexpected DTDValidator error
java.lang.NullPointerException: Cannot invoke "org.eclipse.lemminx.dom.DOMAttr.getNodeAttrName()" because "attr" is null
	at org.eclipse.lemminx.utils.XMLPositionUtility.createAttrNameRange(XMLPositionUtility.java:217)
	at org.eclipse.lemminx.utils.XMLPositionUtility.selectAttributeNameAt(XMLPositionUtility.java:98)
	at org.eclipse.lemminx.extensions.contentmodel.participants.XMLSyntaxErrorCode.toLSPRange(XMLSyntaxErrorCode.java:286)
	at org.eclipse.lemminx.extensions.contentmodel.participants.diagnostics.LSPErrorReporterForXML.toLSPRange(LSPErrorReporterForXML.java:62)
	at org.eclipse.lemminx.extensions.xerces.AbstractReferencedGrammarLSPErrorReporter.toLSPRange(AbstractReferencedGrammarLSPErrorReporter.java:84)
	at org.eclipse.lemminx.extensions.xerces.AbstractLSPErrorReporter.internalToLSPRange(AbstractLSPErrorReporter.java:216)
	at org.eclipse.lemminx.extensions.xerces.AbstractLSPErrorReporter.reportError(AbstractLSPErrorReporter.java:91)
	at org.apache.xerces.impl.XMLErrorReporter.reportError(Unknown Source)
	at org.apache.xerces.impl.XMLErrorReporter.reportError(Unknown Source)
	at org.apache.xerces.impl.XMLScanner.reportFatalError(Unknown Source)
	at org.apache.xerces.impl.XMLScanner.scanAttributeValue(Unknown Source)
	at org.apache.xerces.impl.XMLDTDScannerImpl.scanAttDefaultDecl(Unknown Source)
	at org.apache.xerces.impl.XMLDTDScannerImpl.scanAttlistDecl(Unknown Source)
	at org.apache.xerces.impl.XMLDTDScannerImpl.scanDecls(Unknown Source)
	at org.apache.xerces.impl.XMLDTDScannerImpl.scanDTDExternalSubset(Unknown Source)
	at org.apache.xerces.impl.dtd.XMLDTDLoader.loadGrammar(Unknown Source)
	at org.eclipse.lemminx.extensions.dtd.participants.diagnostics.DTDValidator.doDiagnostics(DTDValidator.java:60)
	at org.eclipse.lemminx.extensions.dtd.participants.diagnostics.DTDDiagnosticsParticipant.doDiagnostics(DTDDiagnosticsParticipant.java:53)
	at org.eclipse.lemminx.services.XMLDiagnostics.doExtensionsDiagnostics(XMLDiagnostics.java:67)
	at org.eclipse.lemminx.services.XMLDiagnostics.doDiagnostics(XMLDiagnostics.java:49)
	at org.eclipse.lemminx.services.XMLLanguageService.doDiagnostics(XMLLanguageService.java:190)
	at org.eclipse.lemminx.services.XMLLanguageService.publishDiagnostics(XMLLanguageService.java:204)
	at org.eclipse.lemminx.XMLTextDocumentService.validate(XMLTextDocumentService.java:716)
	at org.eclipse.lemminx.XMLTextDocumentService.lambda$validate$35(XMLTextDocumentService.java:691)
	at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1804)
	at java.base/java.util.concurrent.CompletableFuture$AsyncRun.exec(CompletableFuture.java:1796)
	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:373)
	at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1182)
	at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1655)
	at java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1622)
	at java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:165)

[angelozerr]

I tested now and I discover the same problem. @ejgutierrez74 vscode-xml should manage your DTD validation like you wished (show error detail instead of just 1 error) but it seems that we have a bug with your DTD

[ejgutierrez74]

Perhaps something related to #945 . vscode-xml could not detect or get the DTD file so it gives the first error....Just an idea...

Im a teacher of computer science in vocational education and this error has been reproduced by different students also

[ejgutierrez74]

Testing your sample files, I found an error in the XML language server:

[Error - 14:12:17] Nov 13, 2023 02:12:17 org.eclipse.lemminx.extensions.dtd.participants.diagnostics.DTDValidator doDiagnostics()
Message: Unexpected DTDValidator error
java.lang.NullPointerException: Cannot invoke "org.eclipse.lemminx.dom.DOMAttr.getNodeAttrName()" because "attr" is null
	at org.eclipse.lemminx.utils.XMLPositionUtility.createAttrNameRange(XMLPositionUtility.java:217)
	at org.eclipse.lemminx.utils.XMLPositionUtility.selectAttributeNameAt(XMLPositionUtility.java:98)
	at org.eclipse.lemminx.extensions.contentmodel.participants.XMLSyntaxErrorCode.toLSPRange(XMLSyntaxErrorCode.java:286)
	at org.eclipse.lemminx.extensions.contentmodel.participants.diagnostics.LSPErrorReporterForXML.toLSPRange(LSPErrorReporterForXML.java:62)
	at org.eclipse.lemminx.extensions.xerces.AbstractReferencedGrammarLSPErrorReporter.toLSPRange(AbstractReferencedGrammarLSPErrorReporter.java:84)
	at org.eclipse.lemminx.extensions.xerces.AbstractLSPErrorReporter.internalToLSPRange(AbstractLSPErrorReporter.java:216)
	at org.eclipse.lemminx.extensions.xerces.AbstractLSPErrorReporter.reportError(AbstractLSPErrorReporter.java:91)
	at org.apache.xerces.impl.XMLErrorReporter.reportError(Unknown Source)
	at org.apache.xerces.impl.XMLErrorReporter.reportError(Unknown Source)
	at org.apache.xerces.impl.XMLScanner.reportFatalError(Unknown Source)
	at org.apache.xerces.impl.XMLScanner.scanAttributeValue(Unknown Source)
	at org.apache.xerces.impl.XMLDTDScannerImpl.scanAttDefaultDecl(Unknown Source)
	at org.apache.xerces.impl.XMLDTDScannerImpl.scanAttlistDecl(Unknown Source)
	at org.apache.xerces.impl.XMLDTDScannerImpl.scanDecls(Unknown Source)
	at org.apache.xerces.impl.XMLDTDScannerImpl.scanDTDExternalSubset(Unknown Source)
	at org.apache.xerces.impl.dtd.XMLDTDLoader.loadGrammar(Unknown Source)
	at org.eclipse.lemminx.extensions.dtd.participants.diagnostics.DTDValidator.doDiagnostics(DTDValidator.java:60)
	at org.eclipse.lemminx.extensions.dtd.participants.diagnostics.DTDDiagnosticsParticipant.doDiagnostics(DTDDiagnosticsParticipant.java:53)
	at org.eclipse.lemminx.services.XMLDiagnostics.doExtensionsDiagnostics(XMLDiagnostics.java:67)
	at org.eclipse.lemminx.services.XMLDiagnostics.doDiagnostics(XMLDiagnostics.java:49)
	at org.eclipse.lemminx.services.XMLLanguageService.doDiagnostics(XMLLanguageService.java:190)
	at org.eclipse.lemminx.services.XMLLanguageService.publishDiagnostics(XMLLanguageService.java:204)
	at org.eclipse.lemminx.XMLTextDocumentService.validate(XMLTextDocumentService.java:716)
	at org.eclipse.lemminx.XMLTextDocumentService.lambda$validate$35(XMLTextDocumentService.java:691)
	at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1804)
	at java.base/java.util.concurrent.CompletableFuture$AsyncRun.exec(CompletableFuture.java:1796)
	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:373)
	at java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1182)
	at java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1655)
	at java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1622)
	at java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:165)

??What do you mean ? That the xml file is wrong ? Its well formed accorded to vs-code, xml-copy editor and netbeans....

[ejgutierrez74]

I tested now and I discover the same problem. @ejgutierrez74 vscode-xml should manage your DTD validation like you wished (show error detail instead of just 1 error) but it seems that we have a bug with your DTD

Also if possible would be nice like XML Copy Editor, from Command Palette or whaever....one instruction to test if the xml file is well formed...and other to Validate XML. No seems that vs-code runs it all together.... Just a suggestion
Thanks for your work

[fbricon]

The NPE is a bug in the language server, we need to fix it. Hopefully, this should restore the DTD validation as you expect it.

Once the DTD bug is fixed, the validation should work OOTB, no need for a specific command. XML errors are visible in the problems view like:

[angelozerr]

@ejgutierrez74 indeed it is a bug from vscode xml. I plan to fix soon

[ejgutierrez74]

But id

The NPE is a bug in the language server, we need to fix it. Hopefully, this should restore the DTD validation as you expect it.

Once the DTD bug is fixed, the validation should work OOTB, no need for a specific command. XML errors are visible in the problems view like:

Thank you very much.
Would it possible or as i said make two different instructions: one to test if a xml file is well formed and other to validate it against DTD file.
Or at least in problems output, make a clear difference ( ideally separated by horizontal line or something like that ) between the xml file ( errors of not well formed file) and errors that came from DTD ( from validation of the xml file against DTD). Because mixing the two diferent source of errors, would be some kind of mess....specially to my students.

Hope this would be fix soon.
Thanks for your effort and work.