Comma divider missing when not all fields present in default skins

[Omertron]

Original issue 1284 created by Omertron on 2010-02-16T13:30:26.000Z:

in the default skin (haven't checked others) the logic for the comma
divider between genres/runtime/certification/language in the details.xml is
incorrect, instead of 'and', 'or' should be used otherwise the comma will
not be used at all unless all fields have data.

So the following line:
<xsl:if test="count(genres) != 0 and runtime != 'UNKNOWN'">, </xsl:if>
Should be
<xsl:if test="count(genres) != 0 or runtime != 'UNKNOWN'">, </xsl:if>

And the following:
<xsl:if test="count(genres) != 0 and runtime != 'UNKNOWN' and certification
!= 'UNKNOWN'">, </xsl:if>
Should be:
<xsl:if test="count(genres) != 0 or runtime != 'UNKNOWN' or certification
!= 'UNKNOWN'">, </xsl:if>

[Omertron]

Comment #1 originally posted by Omertron on 2010-02-16T20:26:03.000Z:

Thanks for that.

Fixed in all skins r1410