Delete a specific confignode

[Diazo]

Alright, this seems like it should be so simple, but I'm drawing a blank.

I have a ConfigNode with 3 sub-nodes that are all named "PART".

As a value on each PART sub-node I have the flightID as a unique value.

I want to delete one of the PART sub-nodes based on this flightID value, how do I go about it?

The closest I've come up with so far is the following:

foreach(ConfigNode nList in masterNode.nodes)
{
if(nList.GetValue("FlightID") == FlightIDOfPartToDelete)
{
masterNode.RemoveNode(nList); //does not work because RemoveNode only takes a string
}
}

So I'm stuck. I'm pretty sure I'm missing a command or method that would allow me to do this but looking at the object browser I'm coming up blank.

Anyone have any ideas?

D.

[lo-fi]

This seems so absurdly simple I'm almost embarrassed to suggest it (particularly given the amount of help you've given me with my code), but... will masterNode.RemoveNode(nList.ToString()); work?

I don't know enough about confignodes to test it myself I'm afraid, I'm still a little baffled by them beyond grabbing stuff out of config files.

Apologies if that's a complete waste of a post, it may have been so obvious you didn't mention having tried it

[Diazo]

Unfortunately not, configNode.RemoveNode(string); removes the first node in the list with that name as string. Your example would not match anything as it would pass the entire node as a string, not just the name.

In my case, all of my nodes are named PART so that would simply delete the first node in the list.

As as work around, I'm currently walking the list, copying all nodes that don't match to a temporary list, deleting all the nodes in the masterNode and then adding them back from the temporary list. (Testing pending, I'm not sure I've got the object linking right yet.)

D.

[lo-fi]

Oh, I see! Hadn't quite understood what was going on there, that makes more sense. Thanks for the explanation

[sarbian]

In mm I use nodes.Remove(subNode)

If you do that you can't iterate on the original list so from memory and no compiler to test

foreach(ConfigNode nList in masterNode.nodes.toList() )

{

if(nList.GetValue("FlightID") == FlightIDOfPartToDelete)

{

masterNode.nodes.Remove(nList); //does not work because RemoveNode only takes a string

}

}

[Diazo]

Wait, I can work standard list operations on ConfigNode.nodes without issue? I was assuming I had to use .addNode and .removeNode methods to avoid causing errors (due to behind the scenes code not exposed).

That makes things much simpler, testing pending of course.

Thanks for that,

D.

[NathanKell]

That is, indeed, all RemoveNode appears to do, anyway.

(It seems to just do RemoveAt() at the first node whose name matches.)