getStaticElementsByTagName() has to check for prefix

rbri · rbri · commit 6d1ef16d8d25 · 2024-12-07T19:39:07.000+01:00
diff --git a/src/main/java/org/htmlunit/html/DomElement.java b/src/main/java/org/htmlunit/html/DomElement.java
@@ -663,7 +663,10 @@ public <E extends HtmlElement> List<E> getStaticElementsByTagName(final String t
         for (final Iterator<HtmlElement> iterator = this.new DescendantHtmlElementsIterator(); iterator.hasNext();) {
             final HtmlElement elem = iterator.next();
             if (elem.getLocalName().equalsIgnoreCase(tagName)) {
-                res.add((E) elem);
+                final String prefix = elem.getPrefix();
+                if (prefix == null || prefix.isEmpty()) {
+                    res.add((E) elem);
+                }
             }
         }
         return res;
diff --git a/src/test/java/org/htmlunit/html/HtmlPage3Test.java b/src/test/java/org/htmlunit/html/HtmlPage3Test.java
@@ -687,4 +687,24 @@ public void getElementById_AfterAppendingToNewlyCreatedElement() throws Exceptio
             + "</body></html>";
         loadPageVerifyTitle2(content);
     }
+
+    /**
+     * When looking for the refresh meta tag don't get confused by stuff with a namespace.
+     * @throws Exception if the test fails
+     */
+    @Test
+    @Alerts("works")
+    public void metaWithNamespace() throws Exception {
+        final String content =
+                "<html>\n"
+                + "<head>\n"
+                + "  <title>works\u00a7</title>\n"
+                + "</head>\n"
+                + "<body>\n"
+                + "  <overheidrg:meta xmlns:overheidrg='http://standaarden.overheid.nl/cvdr/terms/'>\n"
+                + "</body>\n"
+                + "</html>";
+
+        loadPageVerifyTitle2(content);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -663,7 +663,10 @@ public <E extends HtmlElement> List<E> getStaticElementsByTagName(final String t`
`663`	`663`	`for (final Iterator<HtmlElement> iterator = this.new DescendantHtmlElementsIterator(); iterator.hasNext();) {`
`664`	`664`	`final HtmlElement elem = iterator.next();`
`665`	`665`	`if (elem.getLocalName().equalsIgnoreCase(tagName)) {`
`666`		`- res.add((E) elem);`
	`666`	`+ final String prefix = elem.getPrefix();`
	`667`	`+ if (prefix == null \|\| prefix.isEmpty()) {`
	`668`	`+ res.add((E) elem);`
	`669`	`+ }`
`667`	`670`	`}`
`668`	`671`	`}`
`669`	`672`	`return res;`