How to remove nodes from an XML file

Question

I want to remove some elements from an XML file using groovy, however this does not seem to work.

Any ideas how to solve this?

def DeploymentDescriptors = new XmlSlurper().parse("pathToMyXMLFile")
DeploymentDescriptors.NameValuePairs.NameValuePair.each {
if(it.name == 'B')
   it.replaceNode{}
}

XML File

<DeploymentDescriptors>
    <NameValuePairs>
        <NameValuePair>
            <name>A</name>
            <value>Value A</value>
        </NameValuePair>
        <NameValuePair>
            <name>B</name>
            <value>Value B</value>
        </NameValuePair>
        <NameValuePair>
            <name>C</name>
            <value>Value C</value>
        </NameValuePair>
    </NameValuePairs>
</DeploymentDescriptors>

Opal · Accepted Answer · 2014-10-03T10:30:53.473

3

XmlSlurper doesn't change the processed XML until there's a need. If You serialize it You'll see the effect.

Here's working example:

import groovy.xml.StreamingMarkupBuilder
import groovy.xml.XmlUtil
def xml = """
<DeploymentDescriptors>
    <NameValuePairs>
        <NameValuePair>
            <name>A</name>
            <value>Value A</value>
        </NameValuePair>
        <NameValuePair>
            <name>B</name>
            <value>Value B</value>
        </NameValuePair>
        <NameValuePair>
            <name>C</name>
            <value>Value C</value>
        </NameValuePair>
    </NameValuePairs>
</DeploymentDescriptors>"""

def parsed = new XmlSlurper().parseText(xml)
parsed.NameValuePairs.NameValuePair.findAll { it.name.text() == 'B' }*.replaceNode{}
println XmlUtil.serialize(new StreamingMarkupBuilder().bind {
  mkp.yield parsed
} )

edited Oct 03 '14 at 10:30

answered Oct 03 '14 at 09:43

Opal

81,889
28
189
210

Function `find` finds the first value matching the closure condition. So if there are two nodes with name 'B' you will remove only 1. OP used `each`, so i suppose he wanted to remove ALL occurences. – Kasper Ziemianek Oct 03 '14 at 10:18
Thanks @xwid for pointing this out. Replaced `find` with `each` and conditional block. – Opal Oct 03 '14 at 10:25
You should use `findAll` instead of `each`. – Kasper Ziemianek Oct 03 '14 at 10:29
Edited. However, not sure what the real intention of OP was. – Opal Oct 03 '14 at 10:31
OP used word "elements" so I suppose he meant more than 1. – Kasper Ziemianek Oct 03 '14 at 10:33

How to remove nodes from an XML file

1 Answers1