Bug 7141

Summary: The biblio details page in the intranet doesn't work if XSLT is activated and the xsl file contains " "
Product: Koha Reporter: Frédérick Capovilla <fcapovilla>
Component: TemplatesAssignee: Frédérick Capovilla <fcapovilla>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, f.demians, koha.sekjal, m.de.rooy, paul.poulain, wizzyrea
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 16608    
Attachments: Correct a problem when translations contain &nbsp; in MARC21 XSL file.
Signed-off patch
Follow-up patch : Corrects other xsl files so &nbsp; doesn't cause problems.
Signed-off follow-up patch
Second Follow-up : Add nbsb support to all xsl files
Group the 3 patches
Bug 7184 : Changing test to match new formatting of xsl style sheet
Bug 7141 : Changing test to match new formatting of xsl style sheet
Bug 7141 : Changing test to match new formatting of xsl style sheet

Description Frédérick Capovilla 2011-11-02 15:16:40 UTC Comment hidden (obsolete)
Comment 1 Frédérick Capovilla 2011-11-02 15:17:46 UTC
Patch sponsored by the CCSR ( http://www.ccsr.qc.ca ).
Comment 2 Frédéric Demians 2011-11-02 15:42:47 UTC Comment hidden (obsolete)
Comment 3 Frédérick Capovilla 2011-11-02 20:57:53 UTC Comment hidden (obsolete)
Comment 4 Frédéric Demians 2011-11-03 05:47:42 UTC Comment hidden (obsolete)
Comment 5 Marcel de Rooy 2011-11-28 10:59:03 UTC
QA Comment:
Test plan could have been helpful. Now I added &nbsp; somewhere to let Koha break and your patch resolved it.

These patches only add the entity definition of nbsp to xsl files. I only came across this warning: 
[A word of caution: some XML-related tools will try to validate the XSLT stylesheet if they see the DOCTYPE. Since the DTD subset does not include definitions for all the XSLT elements, validation errors will be reported.]

Another point is that this patch adds this definition to a selected set of stylesheets. You could ask that if you want to add this definition, why not do it in all stylesheets? The current selection is based on an assumption that might perhaps not cover all cases.

Although I would not oppose pushing this patch (with these notes), I am asking Ian as QAM to update the status, if he agrees too.
Comment 6 Frédérick Capovilla 2011-11-29 14:58:09 UTC Comment hidden (obsolete)
Comment 7 Frédéric Demians 2011-11-30 08:32:27 UTC Comment hidden (obsolete)
Comment 8 Ian Walls 2011-12-01 12:24:58 UTC
This solution has already been implemented in UNIMARC XSLTs, and MARC21 OPAC XSLT, so implementing it for the rest of the MARC21 style sheets would be consistent.  Marking as Passed QA.
Comment 9 Paul Poulain 2011-12-06 11:19:40 UTC
patch pushed, please test (hugin will be happy ;-) )
Comment 10 Chris Cormack 2011-12-07 20:29:26 UTC Comment hidden (obsolete)
Comment 11 Chris Cormack 2011-12-07 20:30:34 UTC Comment hidden (obsolete)
Comment 12 Liz Rea 2011-12-07 20:42:58 UTC
Created attachment 6644 [details] [review]
Bug 7141 : Changing test to match new formatting of xsl style sheet

http://bugs.koha-community.org/show_bug.cgi?id=7141
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
verified tests pass
Comment 13 Paul Poulain 2011-12-14 13:24:02 UTC
unless i've missed something, the attached patch has been pushed already
Comment 14 Jared Camins-Esakov 2012-05-23 12:27:04 UTC
Included in the 3.6 branch prior to 3.6.4.