Hi Andy,

Thanks for the tip - that makes sense.

I tried this;

<xsl:for-each select="//dc:creatorid | //dc:creator1id | //dc:creator2id | //dc:creator3id"> <component group="authors_logins" name="value"> <xsl:value-of select="."/><xsl:if test="position() != last()">, </xsl:if> </component> </xsl:for-each>

And the output was just this. <component name="value" group="authors_logins">n030, </component>

The metadata has values for dc:creatorid and dc:creator1id so there should be a value after the comma.

I also tried this but again the output is the same.

<xsl:for-each select="//dc:creatorid | //dc:creator1id | //dc:creator2id | //dc:creator3id"> <component group="authors_logins" name="value"> <xsl:value-of select="."/> <xsl:if test="position() != last()"> <xsl:text>, </xsl:text> </xsl:if> </component> </xsl:for-each>

Perhaps it is something to do with the // before the element name?? I tried cominations without these (select="//dc:creatorid | dc:creator1id |) but didn't fix it. Are these paths?

Thanks for your help. Mark

Mark Jones University of East Anglia Norwich NR4 7TJ Tel: 01603 59 1250