2003-02-03 Gonzalo Paniagua Javier <gonzalo@ximian.com>
[mono.git] / mcs / class / System.XML / System.Xml / XmlNamedNodeMap.cs
index c82343f085a35005232680dcdb7d589047be6e7f..756b30d18b6e0206f0675580e57f21a7de408658 100644 (file)
@@ -17,7 +17,7 @@ namespace System.Xml
        {
                XmlNode parent;
                ArrayList nodeList;
-               bool readonly;
+               bool readOnly = false;
 
                internal XmlNamedNodeMap (XmlNode parent)
                {
@@ -31,7 +31,7 @@ namespace System.Xml
 
                public virtual IEnumerator GetEnumerator () 
                {
-                       return ArrayList.GetEnumerator ();
+                       return nodeList.GetEnumerator ();
                }
 
                public virtual XmlNode GetNamedItem (string name)
@@ -46,8 +46,8 @@ namespace System.Xml
                public virtual XmlNode GetNamedItem (string localName, string namespaceURI)
                {
                        foreach (XmlNode node in nodeList) {
-                               if ((node.Name == name)
-                                   && (parent.NamespaceURI == namespaceURI))
+                               if ((node.LocalName == localName)
+                                   && (node.NamespaceURI == namespaceURI))
                                        return node;
                        }
 
@@ -63,38 +63,52 @@ namespace System.Xml
                }
 
                public virtual XmlNode RemoveNamedItem (string name)
-               {
-                       XmlNode removed = null;
-
+               {                       
                        foreach (XmlNode node in nodeList)
                                if (node.Name == name) {
-                                       removed = node;
                                        nodeList.Remove (node);
+                                       return node;
                                }
                        
-                       return removed;
+                       return null;
                }
 
                public virtual XmlNode RemoveNamedItem (string localName, string namespaceURI)
                {
-                       XmlNode removed = null;
-
                        foreach (XmlNode node in nodeList)
-                               if ((node.Name == name)
-                                   && (parent.NamespaceURI == namespaceURI)) {
-                                       removed = node;
+                               if ((node.LocalName == localName)
+                                   && (node.NamespaceURI == namespaceURI)) {
                                        nodeList.Remove (node);
+                                       return node;
                                }
 
-                       return removed;
+                       return null;
                }
 
                public virtual XmlNode SetNamedItem (XmlNode node)
                {
-                       if (readonly || (node.OwnerDocument != parent.OwnerDocument))
+                       return SetNamedItem(node, -1);
+               }
+
+               internal XmlNode SetNamedItem (XmlNode node, int pos)
+               {
+                       if (readOnly || (node.OwnerDocument != parent.OwnerDocument))
                                throw new ArgumentException ("Cannot add to NodeMap.");
-                       nodeList.Add (node);
-                       return node;
+
+                       foreach (XmlNode x in nodeList)
+                               if(x.LocalName == node.LocalName && x.NamespaceURI == node.NamespaceURI) {
+                                       nodeList.Remove (x);
+                                       nodeList.Add (node);
+                                       return x;
+                               }
+                       
+                       if(pos < 0)
+                               nodeList.Add (node);
+                       else
+                               nodeList.Insert(pos, node);
+                       return null;
                }
+
+               internal ArrayList Nodes { get { return nodeList; } }
        }
 }