Bug 6886: (follow-up) replace needless <xsl:choose> with <xsl:if>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes all tests and QA script. Switching SingleBranchMode: Tested with MARC21 and UNIMARC: - "Don't allow" - branch is not shown in result list - "Allow" - branch is shown for available items in result list Changes are consistent over all 3 MARC flavours. Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
parent
a8428e141e
commit
5b855c76d0
|
|
@ -1062,11 +1062,9 @@
|
|||
</xsl:when>
|
||||
<xsl:otherwise>
|
||||
<xsl:for-each select="$available_items[generate-id() = generate-id(key('item-by-status-and-branch', concat(items:status, ' ', items:homebranch))[1])]">
|
||||
<xsl:choose>
|
||||
<xsl:when test="$singleBranchMode=0">
|
||||
<xsl:value-of select="items:homebranch"/>
|
||||
</xsl:when>
|
||||
</xsl:choose>
|
||||
<xsl:if test="$singleBranchMode=0">
|
||||
<xsl:value-of select="items:homebranch"/>
|
||||
</xsl:if>
|
||||
<xsl:if test="items:itemcallnumber != '' and items:itemcallnumber and $OPACItemLocation='callnum'"> [<xsl:value-of select="items:itemcallnumber"/>]</xsl:if>
|
||||
<xsl:text> (</xsl:text>
|
||||
<xsl:value-of select="count(key('item-by-status-and-branch', concat(items:status, ' ', items:homebranch)))"/>
|
||||
|
|
@ -1086,11 +1084,9 @@
|
|||
<b><xsl:text>Copies available for reference: </xsl:text></b>
|
||||
<xsl:variable name="reference_items" select="key('item-by-status', 'reference')"/>
|
||||
<xsl:for-each select="$reference_items[generate-id() = generate-id(key('item-by-status-and-branch', concat(items:status, ' ', items:homebranch))[1])]">
|
||||
<xsl:choose>
|
||||
<xsl:when test="$singleBranchMode=0">
|
||||
<xsl:value-of select="items:homebranch"/>
|
||||
</xsl:when>
|
||||
</xsl:choose>
|
||||
<xsl:if test="$singleBranchMode=0">
|
||||
<xsl:value-of select="items:homebranch"/>
|
||||
</xsl:if>
|
||||
<xsl:if test="items:itemcallnumber != '' and items:itemcallnumber"> [<xsl:value-of select="items:itemcallnumber"/>]</xsl:if>
|
||||
<xsl:text> (</xsl:text>
|
||||
<xsl:value-of select="count(key('item-by-status-and-branch', concat(items:status, ' ', items:homebranch)))"/>
|
||||
|
|
|
|||
|
|
@ -814,12 +814,9 @@
|
|||
<xsl:variable name="reference_items"
|
||||
select="key('item-by-status', 'reference')"/>
|
||||
<xsl:for-each select="$reference_items[generate-id() = generate-id(key('item-by-status-and-branch', concat(items:status, ' ', items:homebranch))[1])]">
|
||||
<xsl:choose>
|
||||
<xsl:when test="$singleBranchMode=0">
|
||||
<xsl:if test="$singleBranchMode=0">
|
||||
<xsl:value-of select="items:homebranch"/>
|
||||
</xsl:when>
|
||||
</xsl:choose>
|
||||
|
||||
</xsl:if>
|
||||
<xsl:if test="items:itemcallnumber != '' and items:itemcallnumber"> [<xsl:value-of select="items:itemcallnumber"/>]</xsl:if>
|
||||
<xsl:text> (</xsl:text>
|
||||
<xsl:value-of select="count(key('item-by-status-and-branch', concat(items:status, ' ', items:homebranch)))"/>
|
||||
|
|
|
|||
|
|
@ -178,11 +178,9 @@
|
|||
<xsl:variable name="reference_items"
|
||||
select="key('item-by-status', 'reference')"/>
|
||||
<xsl:for-each select="$reference_items[generate-id() = generate-id(key('item-by-status-and-branch', concat(items:status, ' ', items:homebranch))[1])]">
|
||||
<xsl:choose>
|
||||
<xsl:when test="$singleBranchMode=0">
|
||||
<xsl:value-of select="items:homebranch"/>
|
||||
</xsl:when>
|
||||
</xsl:choose>
|
||||
<xsl:if test="$singleBranchMode=0">
|
||||
<xsl:value-of select="items:homebranch"/>
|
||||
</xsl:if>
|
||||
<xsl:if test="items:itemcallnumber != '' and items:itemcallnumber">[<xsl:value-of select="items:itemcallnumber"/>]</xsl:if>
|
||||
<xsl:text> (</xsl:text>
|
||||
<xsl:value-of select="count(key('item-by-status-and-branch', concat(items:status, ' ', items:homebranch)))"/>
|
||||
|
|
|
|||
Loading…
Reference in New Issue