[bugfix] Further improvements to coalescing namespace prefixes

adamretter · adamretter · commit cc5f69b87b19 · 2025-08-08T19:09:08.000+02:00
See #30
diff --git a/exist-core/src/main/java/org/exist/util/serializer/SAXSerializer.java b/exist-core/src/main/java/org/exist/util/serializer/SAXSerializer.java
@@ -327,7 +327,7 @@ public void startElement(final QName qname, final AttrList attribs) throws SAXEx
             for (final Map.Entry<String, String> nsEntry : optionalNamespaceDecls.entrySet()) {
                 final String prefix = nsEntry.getKey();
                 final String uri = nsEntry.getValue();
-                if (!(elemPrefixedNsIsDefaultNs && uri.equals(namespaceURI) && elemPrefix.equals(prefix))) {
+                if (!(elemPrefixedNsIsDefaultNs && uri.equals(namespaceURI) && StringUtil.equals(qname.getPrefix(), prefix))) {
                     receiver.namespace(prefix, uri);
                 }
             }
@@ -338,7 +338,7 @@ public void startElement(final QName qname, final AttrList attribs) throws SAXEx
                 }
                 final String uri = nsEntry.getValue();
                 if(!optionalNamespaceDecls.containsKey(prefix)) {
-                    if (!(elemPrefixedNsIsDefaultNs && uri.equals(namespaceURI) && elemPrefix.equals(prefix))) {
+                    if (!(elemPrefixedNsIsDefaultNs && uri.equals(namespaceURI) && StringUtil.equals(qname.getPrefix(), prefix))) {
                         receiver.namespace(prefix, uri);
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -327,7 +327,7 @@ public void startElement(final QName qname, final AttrList attribs) throws SAXEx`
`327`	`327`	`for (final Map.Entry<String, String> nsEntry : optionalNamespaceDecls.entrySet()) {`
`328`	`328`	`final String prefix = nsEntry.getKey();`
`329`	`329`	`final String uri = nsEntry.getValue();`
`330`		`- if (!(elemPrefixedNsIsDefaultNs && uri.equals(namespaceURI) && elemPrefix.equals(prefix))) {`
	`330`	`+ if (!(elemPrefixedNsIsDefaultNs && uri.equals(namespaceURI) && StringUtil.equals(qname.getPrefix(), prefix))) {`
`331`	`331`	`receiver.namespace(prefix, uri);`
`332`	`332`	`}`
`333`	`333`	`}`
`@@ -338,7 +338,7 @@ public void startElement(final QName qname, final AttrList attribs) throws SAXEx`
`338`	`338`	`}`
`339`	`339`	`final String uri = nsEntry.getValue();`
`340`	`340`	`if(!optionalNamespaceDecls.containsKey(prefix)) {`
`341`		`- if (!(elemPrefixedNsIsDefaultNs && uri.equals(namespaceURI) && elemPrefix.equals(prefix))) {`
	`341`	`+ if (!(elemPrefixedNsIsDefaultNs && uri.equals(namespaceURI) && StringUtil.equals(qname.getPrefix(), prefix))) {`
`342`	`342`	`receiver.namespace(prefix, uri);`
`343`	`343`	`}`
`344`	`344`	`}`