views:

3339

answers:

5

The following code should find the appropriate project tag and remove it from the XmlDocument, however when I test it, it says:

The node to be removed is not a child of this node.

Does anyone know the proper way to do this?

public void DeleteProject (string projectName)
{
    string ccConfigPath = ConfigurationManager.AppSettings["ConfigPath"];

    XmlDocument configDoc = new XmlDocument();

    configDoc.Load(ccConfigPath);

    XmlNodeList projectNodes = configDoc.GetElementsByTagName("project");

    for (int i = 0; i < projectNodes.Count; i++)
    {
        if (projectNodes[i].Attributes["name"] != null)
        {
            if (projectName == projectNodes[i].Attributes["name"].InnerText)
            {                                                
                configDoc.RemoveChild(projectNodes[i]);
                configDoc.Save(ccConfigPath);
            }
        }
    }
}

UPDATE

Fixed. I did two things:

XmlNode project = configDoc.SelectSingleNode("//project[@name='" + projectName + "']");

Replaced the For loop with an XPath query, which wasn't for fixing it, just because it was a better approach.

The actual fix was:

project.ParentNode.RemoveChild(project);

Thanks Pat and Chuck for this suggestion.

+1  A: 

Is it possible that the project nodes aren't child nodes, but grandchildren or lower? GetElementsByTagName will give you elements from anywhere in the child element tree, IIRC.

Greg Hurlman
A: 

It would be handy to see a sample of the XML file you're processing but my guess would be that you have something like this

<Root>
 <Blah>
   <project>...</project>
 </Blah>
</Root>

The error message seems to be because you're trying to remove from the grandparent rather than the direct parent of the project node

David Hayes
+8  A: 

Instead of

configDoc.RemoveChild(projectNodes[i]);

try

projectNodes[i].parentNode.RemoveChild(projectNodes[i]);
Pat
A: 

try

configDoc.DocumentElement.RemoveChild(projectNodes[i]);
Jason
+1  A: 

Looks like you need to select the parent node of projectNodes[i] before calling RemoveChild.

Chuck