Bug 14326

Summary: XSLT Syntax error in MARC21slimOPACResults.xsl
Product: Koha Reporter: Winona Salesky <wsalesky>
Component: TemplatesAssignee: Winona Salesky <wsalesky>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: chris, jonathan.druart, katrin.fischer, liz, mtj, nicole, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 4421    
Bug Blocks:    
Attachments: Bug 14326 - XSLT Syntax error in MARC21slimOPACResults.xsl
[SIGNED OFF] Bug 14326 - XSLT Syntax error in MARC21slimOPACResults.xsl
Bug 14326 - XSLT Syntax error in MARC21slimOPACResults.xsl

Description Winona Salesky 2015-06-04 02:45:16 UTC
In MARC21slimOPACResults.xsl on line 1083, there is a xsl:call-template with a select in addition to a name attribute. Select can not be used with xsl:call-template. This throws an error in Saxon and is invalid xslt. As far as I can tell this select can simply be removed as the xsl:for-each parent statement is using the same logic. 

-Winona
Comment 1 Winona Salesky 2015-06-04 03:03:43 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2015-06-06 16:20:08 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-06-23 11:00:22 UTC
Created attachment 40518 [details] [review]
Bug 14326 - XSLT Syntax error in MARC21slimOPACResults.xsl

Test Plan:
1) Apply this patch
2) Ensure you are using the default XSLT setting for the staff and opac  record details
3) Perform an opac search check "Availability" for expected display values.
5) Note this patch corrects invalid syntax in xslt, there should be no visable changes to the results page.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Comment 4 Jonathan Druart 2015-06-23 11:00:46 UTC
Introduced by bug 4421.
Comment 5 Tomás Cohen Arazi 2015-07-08 17:42:23 UTC
Patch pushed to master.

Thanks Winona!
Comment 6 Chris Cormack 2015-07-14 08:35:37 UTC
Pushed to 3.20.x will be in 3.20.2
Comment 7 Liz Rea 2015-07-20 22:16:00 UTC
Pushed to 3.18.x will be in 3.18.09
Comment 8 Mason James 2015-07-25 13:07:55 UTC
Pushed to 3.16.x, will be in 3.16.13