Revisted a bug, and fixed another
Posted by mholmes on 02 Apr 2007 in Activity log
The bug fix described here turned out to have been inadequate; that code has now been replaced with this:
<xsl:choose> <xsl:when test="$PutDateAfterAuthor = 'true'"> <xsl:if test="(string-length($TheParent/imprint) > 0) and (string-length($TheParent/imprint) != string-length($TheParent/imprint/date))"> <xsl:text>. </xsl:text> </xsl:if> </xsl:when> <xsl:otherwise> <xsl:if test="(string-length($TheParent/imprint) > 0)"><xsl:text>. </xsl:text></xsl:if> </xsl:otherwise> </xsl:choose>
In addition, I noticed that when an article title in a reference ended in a question mark, the following part of the reference ran on immediately, without an intervening space. To fix this, I changed this:
<xsl:if test="not(substring($InTitle/text(), string-length($InTitle/text()))='?')"><xsl:text>. </xsl:text></xsl:if>
to this:
<xsl:if test="not(substring($InTitle/text(), string-length($InTitle/text()))='?')"><xsl:text>.</xsl:text></xsl:if><xsl:text> </xsl:text>
There is still a potential problem with titles ending in exclamation marks, though. We may have to come back to this.
All of these changes were made in both the XHTML and PDF rendering libraries.