Bug 7394

Summary: Broken detail page for last link from result page
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: OPACAssignee: Chris Cormack <chris>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: critical    
Priority: PATCH-Sent (DO NOT USE) CC: arslanone, chris, cnighswonger, f.demians, paul.poulain, salva
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 6483    
Bug Blocks:    
Attachments: wrong display on opac detail page
Bug 7394 : Fixing scoping issue with XSLT processing. Also bailing out early for 404 inestead of doing unnessecary processing
Signed-off patch
[SIGNED-OFF] Bug 7394 : Fixing scoping issue with XSLT processing. Also bailing out early for 404 inestead of doing unnessecary processing
[SIGNED-OFF] Bug 7394 : Fixing scoping issue with XSLT processing. Also bailing out early for 404 inestead of doing unnessecary processing

Description Katrin Fischer 2012-01-02 14:01:58 UTC
Created attachment 7013 [details]
wrong display on opac detail page

Found a weird bug, where the detail page wrongly uses the result page XSLT.
The title shows formatted as link and availability information is displayed. (see screenshot)

To reproduce:

1) OPACXSLTDetailsDisplay = ON
2) OPACXSLTResultDisplay = ON
3) Search for anything with more than 20 results. 
4) Open the last record (normally 20th) of your the result page.
5) Go to the second reult page, same happens here for the last link (40th)
Comment 1 Chris Cormack 2012-01-03 06:47:44 UTC
Its the browse results thing that is causing this.

Which reminds me, that whole thing should be wrapped in a system preference, as well as fixed
Comment 2 Chris Cormack 2012-01-03 06:58:57 UTC Comment hidden (obsolete)
Comment 3 Chris Cormack 2012-01-03 07:00:15 UTC
Fix for problem, will do 2 follow ups

1 to add a syspref, 1 to shift all the code to a module instead of in the opac-detail.pl

But please signoff and push the fix without waiting on the follow ups
Comment 4 Frédéric Demians 2012-01-03 07:09:34 UTC
Related to bug 6483.
Comment 5 Frédéric Demians 2012-01-03 07:16:44 UTC Comment hidden (obsolete)
Comment 6 Katrin Fischer 2012-01-03 07:16:58 UTC Comment hidden (obsolete)
Comment 7 Katrin Fischer 2012-01-03 07:20:07 UTC
Created attachment 7024 [details] [review]
[SIGNED-OFF] Bug 7394 : Fixing scoping issue with XSLT processing. Also bailing out early for 404 inestead of doing unnessecary processing

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>

I can confirm the bug and the solution.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Second sign-off from me, patch fixes display of detail page
for last record on a result page.
Comment 8 Ian Walls 2012-01-03 18:31:57 UTC
Moves code to different area of script to avoid scoping issues.  Fixes issue without introducing any new complexities.  Marking as Passed QA
Comment 9 Paul Poulain 2012-01-06 11:34:38 UTC
patch pushed, pleast test
Comment 10 Chris Nighswonger 2012-01-21 04:37:28 UTC
This patch does not apply cleanly to 3.4.x. Please rebase and attach a new patch.
Comment 11 Katrin Fischer 2012-01-21 07:47:18 UTC
The 'browse results' was a new feature in 3.6 and is not in 3.4 (afaik). The problem should not appear there.
Comment 12 Chris Nighswonger 2012-01-21 12:42:24 UTC
TNX!