Bug 5545

Summary: links encoding wrong on XSLT search results
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: CLOSED INVALID QA Contact: Bugs List <koha-bugs>
Severity: major    
Priority: P5 - low CC: helen.linda
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:

Description Nicole C. Engard 2010-12-27 20:52:07 UTC
With XSLT on when a link appears on the search results it encodes amperstands and special characters in ASCII - causing broken links.

Example:

The link is:

http://mylibrary.edu:2048/login?url=http://www.ebookssite.com/urlapi.asp?action=summary&v=1&bookid=92989

But when you click from the results list the link is:

http://mylibrary.edu:2048/login?url=http://www.ebookssite.com/urlapi.asp?action=summary&amp;v=1&amp;bookid=92989

The link is A-OK on the detail page, it's just the results listing that has the encoding wrong.
Comment 1 Nicole C. Engard 2010-12-28 16:17:53 UTC
[10:56]  <owen> nengard: Bug 5545 happens when a record has an 856?
[10:56]  <munin> Bug
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5545 major, P5, ---,
oleonard, NEW, links encoding wrong on XSLT search results
[10:56]  <nengard> yes
[11:08]  <owen> nengard: I'm not seeing the problem
[11:08]  <nengard> hmmmm
[11:09]  <nengard> hang on
[11:09]  <nengard> owen check here:
http://catalog.goddard.edu/cgi-bin/koha/opac-search.pl?idx=kw&q=education&idx=kw&idx=kw&do=Search&limit=mc-itype%3AEBOOK&sort_by=relevance 
[11:10]  <nengard> and then check any bib record on that list
[11:10]  <nengard> the link on the results has an &amp
[11:10]  <nengard> but the link on the bib detail is right
[11:13]  <owen> nengard: Can you replicate that problem in your test system? I
can't. Maybe I'm doing it wrong
[11:13]  <nengard> trying now
[11:13]  <nengard> my test system isn't showing links on the results at all ... 
[11:13]  <nengard> might have to do a rebuild
[11:15]  <nengard> hmmm - rebuild done and you are right, time to poke into the
data on that other system and see what's going on