Summary: | OPAC results and lists pages contain invalid attributes (xmlns:str="http://exslt.org/strings") | ||
---|---|---|---|
Product: | Koha | Reporter: | Henry Bolshaw <bolshawh> |
Component: | OPAC | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | andrew, caroline.cyr-la-rose, fridolin.somers, jonathan.druart, m.de.rooy, martin.renvoize, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
21.05.00,20.11.02,20.05.08
|
|
Circulation function: | |||
Bug Depends on: | 14441 | ||
Bug Blocks: | |||
Attachments: |
Bug 27178: Add 'str' namespace to exclude-result-prefixes
Bug 27178: Add 'str' namespace to exclude-result-prefixes Bug 27178: Add 'str' namespace to exclude-result-prefixes |
Description
Henry Bolshaw
2020-12-09 14:52:48 UTC
It might come from a exslt change. If I have only the "include" in MARC21slim2OPACResults.xsl I am seeing different nodes getting this attribute. xmlns:str="http://exslt.org/strings" Created attachment 114298 [details] [review] Bug 27178: Add 'str' namespace to exclude-result-prefixes Some nodes in the generated HTML document contain the 'str' namespace. For instance, at the OPAC on the search result list, you can see: <a xmlns:str="http://exslt.org/strings" href="/cgi-bin/koha/opac-detail.pl?biblionumber=18566&query_desc=kw%2Cwrdl%3A%20spin" class="title"> The solution is to add 'str' to exclude-result-prefixes. From https://developer.mozilla.org/en-US/docs/Web/XSLT/Element/stylesheet """ Specifies any namespace used in this document that should not be sent to the output document. The list is whitespace separated. """ Test plan: 0. Don't apply this patch 1. Launch a search at the OPAC 2. Inspect the title link for the records => Note the 'xmlns:str' attribute on tag "a" 3. Apply this patch 4. restart_all 5. Repeat 1. and 2. => Note that the attribute is gone now. QA will check that no other occurrences is missing (note that one NORMARC xsl is missing it, but NORMARC is deprecated) Created attachment 114308 [details] [review] Bug 27178: Add 'str' namespace to exclude-result-prefixes Some nodes in the generated HTML document contain the 'str' namespace. For instance, at the OPAC on the search result list, you can see: <a xmlns:str="http://exslt.org/strings" href="/cgi-bin/koha/opac-detail.pl?biblionumber=18566&query_desc=kw%2Cwrdl%3A%20spin" class="title"> The solution is to add 'str' to exclude-result-prefixes. From https://developer.mozilla.org/en-US/docs/Web/XSLT/Element/stylesheet """ Specifies any namespace used in this document that should not be sent to the output document. The list is whitespace separated. """ Test plan: 0. Don't apply this patch 1. Launch a search at the OPAC 2. Inspect the title link for the records => Note the 'xmlns:str' attribute on tag "a" 3. Apply this patch 4. restart_all 5. Repeat 1. and 2. => Note that the attribute is gone now. QA will check that no other occurrences is missing (note that one NORMARC xsl is missing it, but NORMARC is deprecated) Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Great work, well found Jonathan! Created attachment 114684 [details] [review] Bug 27178: Add 'str' namespace to exclude-result-prefixes Some nodes in the generated HTML document contain the 'str' namespace. For instance, at the OPAC on the search result list, you can see: <a xmlns:str="http://exslt.org/strings" href="/cgi-bin/koha/opac-detail.pl?biblionumber=18566&query_desc=kw%2Cwrdl%3A%20spin" class="title"> The solution is to add 'str' to exclude-result-prefixes. From https://developer.mozilla.org/en-US/docs/Web/XSLT/Element/stylesheet """ Specifies any namespace used in this document that should not be sent to the output document. The list is whitespace separated. """ Test plan: 0. Don't apply this patch 1. Launch a search at the OPAC 2. Inspect the title link for the records => Note the 'xmlns:str' attribute on tag "a" 3. Apply this patch 4. restart_all 5. Repeat 1. and 2. => Note that the attribute is gone now. QA will check that no other occurrences is missing (note that one NORMARC xsl is missing it, but NORMARC is deprecated) Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Pushed to master for 21.05, thanks to everybody involved! Pushed to 20.11.x for 20.11.02 Pushed to 20.05.x for 20.05.8 Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed. |