Fix completion & hover based on XSD with

elementFormDefault="unqualified"

See redhat-developer/vscode-xml#311

Signed-off-by: azerr <[email protected]>

org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/extensions/contentmodel/model/ContentModelManager.java

@@ -68,9 +68,9 @@ public ContentModelManager(URIResolverExtensionManager resolverManager) {
 	/**
 	 * Returns the owner document of the declared element which matches the given
 	 * XML element and null otherwise.
-	 * 
+	 *
 	 * @param element the XML element
-	 * 
+	 *
 	 * @return the owner document of the declared element which matches the given
 	 *         XML element and null otherwise.
 	 */
@@ -81,9 +81,9 @@ public Collection<CMDocument> findCMDocument(DOMElement element) {
 	/**
 	 * Returns the owner document of the declared element which matches the given
 	 * XML element and null otherwise.
-	 * 
+	 *
 	 * @param element the XML element
-	 * 
+	 *
 	 * @return the owner document of the declared element which matches the given
 	 *         XML element and null otherwise.
 	 */
@@ -97,12 +97,30 @@ public Collection<CMDocument> findCMDocument(DOMDocument xmlDocument, String nam
 
 	/**
 	 * Returns the declared documents which match the given DOM document.
-	 * 
+	 *
 	 * @param xmlDocument  the DOM document.
 	 * @param namespaceURI the namespace URI
 	 * @return the declared documents which match the given DOM document.
 	 */
 	public Collection<CMDocument> findCMDocument(DOMDocument xmlDocument, String namespaceURI, boolean withInternal) {
+		if (namespaceURI == null) {
+			// This case comes from when an element has no namespace and XML Schema defines
+			// elementFormDefault="unqualified"
+			// --> we use the namespace from the DOM document
+
+			// ex: XSD:
+			// <xs:schema targetNamespace="urn:reports/itops"
+			// elementFormDefault="unqualified"
+
+			// ex : XML
+			// <i:ITOpsReport xmlns:i="urn:reports/itops"
+			// xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+			// xsi:schemaLocation="urn:reports/itops ../schema/reports/itops.xsd">
+			// <templates> --> here template has a null namespace and the DOM document has
+			// "urn:reports/itops" namespace
+			// --> we use urn:reports/itops as namespace to get the DOM Document
+			namespaceURI = xmlDocument.getNamespaceURI();
+		}
 		Collection<CMDocument> documents = new ArrayList<>();
 		for (ContentModelProvider modelProvider : modelProviders) {
 			// internal grammar
@@ -142,7 +160,7 @@ public Collection<CMDocument> findCMDocument(DOMDocument xmlDocument, String nam
 	/**
 	 * Returns true if the given document is linked to the given grammar URI (XML
 	 * Schema, DTD) and false otherwise.
-	 * 
+	 *
 	 * @param document   the DOM document
 	 * @param grammarURI the grammar URI
 	 * @return true if the given document is linked to the given grammar URI (XML
@@ -172,7 +190,7 @@ public boolean dependsOnGrammar(DOMDocument document, String grammarURI) {
 	/**
 	 * Returns the content model document loaded by the given uri and null
 	 * otherwise.
-	 * 
+	 *
 	 * @param publicId      the public identifier.
 	 * @param systemId      the expanded system identifier.
 	 * @param modelProvider
@@ -248,9 +266,9 @@ private void cache(String key, CMDocument cmDocument) {
 
 	/**
 	 * Returns the model provider by the given uri and null otherwise.
-	 * 
+	 *
 	 * @param uri the grammar URI
-	 * 
+	 *
 	 * @return the model provider by the given uri and null otherwise.
 	 */
 	public ContentModelProvider getModelProviderByURI(String uri) {
@@ -264,7 +282,7 @@ public ContentModelProvider getModelProviderByURI(String uri) {
 
 	/**
 	 * Set up XML catalogs.
-	 * 
+	 *
 	 * @param catalogs list of XML catalog files.
 	 * @return true if catalogs changed and false otherwise
 	 */
@@ -281,7 +299,7 @@ public void refreshCatalogs() {
 
 	/**
 	 * Set file associations.
-	 * 
+	 *
 	 * @param fileAssociations
 	 * @return true if file associations changed and false otherwise
 	 */

org.eclipse.lemminx/src/test/java/org/eclipse/lemminx/extensions/contentmodel/XMLSchemaCompletionExtensionsTest.java

@@ -1089,6 +1089,19 @@ public void generateOnlyStartElementOnText() throws BadLocationException {
 				c("employee", te(3, 0, 3, 0, "<employee>$1$0"), "employee"));
 	}
 
+	@Test
+	public void completionWithUnqualifiedElementFormDefault() throws BadLocationException {
+		String xml = "<f:foo xmlns:f=\"http://foo\"\r\n" + //
+				"	xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\"\r\n" + //
+				"	xsi:schemaLocation=\"\r\n" + //
+				"		http://foo xsd/foo-unqualified.xsd\">\r\n" + //
+				"	<bar>\r\n" + //
+				"		|";
+		XMLAssert.testCompletionFor(xml, null, "src/test/resources/foo-unqualified.xml", //
+				null, //
+				c("item", te(5, 2, 5, 2, "<item></item>"), "item", null, null));
+	}
+
 	@Test
 	public void generateOnlyStartElementOnElement() throws BadLocationException {
 		// </employee> already exists, completion must generate only <employee>

org.eclipse.lemminx/src/test/resources/xml/foo-unqualified.xml

@@ -0,0 +1,8 @@
+<f:foo xmlns:f="http://foo"
+	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+	xsi:schemaLocation="
+		http://foo ../xsd/foo-unqualified.xsd">
+	<bar>
+		<item></item>
+	</bar>
+</f:foo>

org.eclipse.lemminx/src/test/resources/xsd/foo-unqualified.xsd

@@ -0,0 +1,19 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<xs:schema elementFormDefault="unqualified"
+	xmlns:xs="http://www.w3.org/2001/XMLSchema"
+	targetNamespace="http://foo">
+	<xs:element name="foo">
+		<xs:complexType>
+			<xs:sequence>
+				<xs:element name="bar">
+					<xs:complexType>
+						<xs:sequence>
+							<xs:element name="item">
+							</xs:element>
+						</xs:sequence>
+					</xs:complexType>
+				</xs:element>
+			</xs:sequence>
+		</xs:complexType>
+	</xs:element>
+</xs:schema>