- my $child = $node->firstChild();
- ### Next line fails if data contains a comment ... *sigh*
- die "element child $child is not text"
- if !ref $child || !$child->isa("XML::LibXML::Text");
- if ($value ne $child->getData()) {
- $child->setData($value);
- $nchanges++;
- print "Elem $key: '", $child->getData(), "' -> '$value' ($xpath)<br/>\n";
+ # The contents could be any mixture of text and
+ # comments and maybe even other crud such as processing
+ # instructions. The simplest thing is just to throw it all
+ # away and start again, making a single Text node the
+ # canonical representation. But before we do that,
+ # we'll check whether the element is already
+ # canonical, to determine whether our change is a
+ # no-op.
+ my $old = "???";
+ my @children = $node->childNodes();
+ if (@children == 1) {
+ my $child = $node->firstChild();
+ if (ref $child && ref $child eq "XML::LibXML::Text") {
+ $old = $child->getData();
+ next if $value eq $old;
+ }