Hyperlink to open declared DTD files

Fixes eclipse#641 Signed-off-by: azerr <[email protected]>
angelozerr · Apr 17, 2020 · 7a013f1 · 7a013f1
1 parent 24cd010
commit 7a013f1
Show file tree

Hide file tree

Showing 2 changed files with 57 additions and 30 deletions.
diff --git a/org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/dom/NoNamespaceSchemaLocation.java b/org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/dom/NoNamespaceSchemaLocation.java
@@ -12,21 +12,15 @@
  */
 package org.eclipse.lemminx.dom;
 
-import org.apache.xerces.impl.XMLEntityManager;
-import org.apache.xerces.util.URI.MalformedURIException;
-
 /**
  * 
  * The declared "xsi:noNamespaceSchemaLocation"
  */
 public class NoNamespaceSchemaLocation {
 
- private final String documentURI;
-
  private final DOMAttr attr;
 
- public NoNamespaceSchemaLocation(String documentURI, DOMAttr attr) {
- this.documentURI = documentURI;
+ public NoNamespaceSchemaLocation(DOMAttr attr) {
  this.attr = attr;
  }
 
@@ -45,21 +39,4 @@ public String getLocation() {
  return attr.getValue();
  }
 
- /**
- * Returns the expanded system location
- * 
- * @return the expanded system location
- */
- public String getResolvedLocation() {
- return getResolvedLocation(documentURI, getLocation());
- }
-
- private String getResolvedLocation(String documentURI, String location) {
- try {
- return XMLEntityManager.expandSystemId(location, documentURI, false);
- } catch (MalformedURIException e) {
- return location;
- }
- }
-
 }
diff --git a/...pse/lemminx/extensions/contentmodel/participants/ContentModelDocumentLinkParticipant.java b/...pse/lemminx/extensions/contentmodel/participants/ContentModelDocumentLinkParticipant.java
@@ -1,5 +1,5 @@
 /**
- * Copyright (c) 2018 Angelo ZERR
+ * Copyright (c) 2018-2020 Angelo ZERR
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v2.0
  * which accompanies this distribution, and is available at
@@ -14,31 +14,81 @@
 
 import java.util.List;
 
+import org.apache.xerces.impl.XMLEntityManager;
+import org.apache.xerces.util.URI.MalformedURIException;
 import org.eclipse.lemminx.commons.BadLocationException;
 import org.eclipse.lemminx.dom.DOMDocument;
+import org.eclipse.lemminx.dom.DOMDocumentType;
 import org.eclipse.lemminx.dom.DOMNode;
+import org.eclipse.lemminx.dom.DOMRange;
 import org.eclipse.lemminx.dom.NoNamespaceSchemaLocation;
 import org.eclipse.lemminx.services.extensions.IDocumentLinkParticipant;
 import org.eclipse.lsp4j.DocumentLink;
 import org.eclipse.lsp4j.Position;
 import org.eclipse.lsp4j.Range;
 
+/**
+ * Document link for :
+ * 
+ * <ul>
+ * <li>XML Schema xsi:noNamespaceSchemaLocation</li>
+ * <li>DTD SYSTEM (ex : <!DOCTYPE root-element SYSTEM "./extended.dtd" )</li>
+ * </ul>
+ * 
+ * @author Angelo ZERR
+ *
+ */
 public class ContentModelDocumentLinkParticipant implements IDocumentLinkParticipant {
 
  @Override
  public void findDocumentLinks(DOMDocument document, List<DocumentLink> links) {
+ // Document link for xsi:noNamespaceSchemaLocation
  NoNamespaceSchemaLocation noNamespaceSchemaLocation = document.getNoNamespaceSchemaLocation();
  if (noNamespaceSchemaLocation != null) {
  try {
- String location = noNamespaceSchemaLocation.getResolvedLocation();
- DOMNode attrValue = noNamespaceSchemaLocation.getAttr().getNodeAttrValue();
- Position start = document.positionAt(attrValue.getStart() + 1);
- Position end = document.positionAt(attrValue.getEnd() - 1);
- links.add(new DocumentLink(new Range(start, end), location));
+ String location = getResolvedLocation(document.getDocumentURI(),
+ noNamespaceSchemaLocation.getLocation());
+ if (location != null) {
+ DOMNode attrValue = noNamespaceSchemaLocation.getAttr().getNodeAttrValue();
+ Position start = document.positionAt(attrValue.getStart() + 1);
+ Position end = document.positionAt(attrValue.getEnd() - 1);
+ links.add(new DocumentLink(new Range(start, end), location));
+ }
  } catch (BadLocationException e) {
  // Do nothing
  }
  }
+ // Document link for DTD
+ DOMDocumentType docType = document.getDoctype();
+ if (docType != null) {
+ String location = getResolvedLocation(document.getDocumentURI(), docType.getSystemIdWithoutQuotes());
+ if (location != null) {
+ try {
+ DOMRange sytemIdRange = docType.getSystemIdNode();
+ Position start = document.positionAt(sytemIdRange.getStart() + 1);
+ Position end = document.positionAt(sytemIdRange.getEnd() - 1);
+ links.add(new DocumentLink(new Range(start, end), location));
+ } catch (BadLocationException e) {
+ // Do nothing
+ }
+ }
+ }
+ }
+
+ /**
+ * Returns the expanded system location
+ * 
+ * @return the expanded system location
+ */
+ private static String getResolvedLocation(String documentURI, String location) {
+ if (location == null) {
+ return null;
+ }
+ try {
+ return XMLEntityManager.expandSystemId(location, documentURI, false);
+ } catch (MalformedURIException e) {
+ return location;
+ }
  }
 
 }