Do not put the xml namespace in a NamespaceMap

ndw · Dec 6, 2022 · bffa7bd · bffa7bd
1 parent 626d767
commit bffa7bd
Showing 1 changed file with 2 additions and 14 deletions.
diff --git a/src/main/java/com/xmlcalabash/util/S9apiUtils.java b/src/main/java/com/xmlcalabash/util/S9apiUtils.java
@@ -323,32 +323,20 @@ private static void removeNamespacesWriter(TreeWriter tree, XdmNode node, HashSe
 
                 changed = changed || delete;
 
-                if (!delete) {
+                // Never put the xml namespace in the NamespaceMap
+                if (!delete && !"xml".equals(pfx)) {
                     newNS.add(new NamespaceBinding(pfx, uri));
                 }
             }
 
             NodeInfo inode = node.getUnderlyingNode();
             AttributeMap attrs = inode.attributes();
-            ArrayList<AttributeInfo> attrList = new ArrayList<> ();
             NodeName newName = NameOfNode.makeName(inode);
             if (!preserveUsed) {
                 NamespaceBinding binding = newName.getNamespaceBinding();
                 if (excludeNS.contains(binding.getURI())) {
                     newName = new FingerprintedQName("", "", newName.getLocalPart());
                 }
-
-                // In this case, we may need to change some attributes too
-                for (AttributeInfo attr : attrs.asList()) {
-                    String attrns = attr.getNodeName().getURI();
-                    if (excludeNS.contains(attrns)) {
-                        FingerprintedQName newAttrName = new FingerprintedQName("", "", attr.getNodeName().getLocalPart());
-                        AttributeInfo newAttr = new AttributeInfo(newAttrName, attr.getType(), attr.getValue(), attr.getLocation(), attr.getProperties());
-                        attrList.add(newAttr);
-                    } else {
-                        attrList.add(attr);
-                    }
-                }
             }
 
             tree.addStartElement(newName, attrs, inode.getSchemaType(), new NamespaceMap(newNS));