Your problem is that you are using the !=
operator for comparison between a value and a node-set.
This is wrong -- always avoid using the !=
operator and always use the not()
function and the =
operator when one of the operands in the comparison is a node-set.
Below is a correct solution:
<xsl:stylesheet version="1.0"
xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
<xsl:output omit-xml-declaration="yes" indent="yes"/>
<xsl:strip-space elements="*"/>
<xsl:template match="/*">
<xsl:for-each select="parent">
<xsl:for-each select="child[not(name = preceding::name)]">
<xsl:value-of select="concat(name, ' ')"/>
</xsl:for-each>
</xsl:for-each>
</xsl:template>
</xsl:stylesheet>
when this transformation is applied on the provided XML document:
<root>
<parent>
<child>
<name>John</name>
</child>
<child>
<name>Ben</name>
</child>
</parent>
<parent>
<child>
<name>John</name>
</child>
<child>
<name>Mark</name>
</child>
<child>
<name>Luke</name>
</child>
</parent>
</root>
the wanted, correct result is produced:
John Ben Mark Luke
Explanation: Here is how the W3C XPath 1.0 spec defines the semantics of the !=
operator:
"If one object to be compared is a
node-set and the other is a string,
then the comparison will be true if
and only if there is a node in the
node-set such that the result of
performing the comparison on the
string-value of the node and the other
string is true."
This means that
's' != node-set
is always true if there is even only one node in node-set
that isn't equal to 's'
.
This isn't the semantics that is wanted.
On the other side,
not('s' = node-set())
is true only only if there isn't a node in node-set
that is equal to 's'
.
This is exactly the wanted comparison.
Do note: The grouping technique you have chosen is O(N^2) and should only be used on very small sets of values to be dedupped. If efficiency is needed, by all means use the Muenchian method for grouping (discussing or demo-ing this falls outside the scope of this question).