Bug 8405 - add IntranetBiblioDefaultView and BiblioDefaultView to XSLT.pm
Summary: add IntranetBiblioDefaultView and BiblioDefaultView to XSLT.pm
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: 3.10
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Gaetan Boisson
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-10 09:12 UTC by Gaetan Boisson
Modified: 2013-05-23 06:24 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
proposed patch (1001 bytes, patch)
2012-07-10 09:21 UTC, Gaetan Boisson
Details | Diff | Splinter Review
Bug 8405 : add IntranetBiblioDefaultView and BiblioDefaultView in XSLT.pm (1.08 KB, patch)
2012-08-01 12:58 UTC, Jared Camins-Esakov
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Gaetan Boisson 2012-07-10 09:12:20 UTC
When displaying the result lists with xslt, the link to the detailed bibliographic record is hardcoded to the default view (detail.pl or opac-detail.pl) and there is no way to accomodate for the syspref value since it is not taken into account in XSLT.pm.

We will submit a patch so that we can test against the value of this syspref in our xslts.
 
IntranetBiblioDefaultView and BiblioDefaultView will be added to XSLT.pm.
Comment 1 Gaetan Boisson 2012-07-10 09:21:52 UTC Comment hidden (obsolete)
Comment 2 Jared Camins-Esakov 2012-08-01 12:58:04 UTC
Created attachment 11252 [details] [review]
Bug 8405 : add IntranetBiblioDefaultView and BiblioDefaultView in XSLT.pm

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Comment 3 Jonathan Druart 2012-08-01 13:45:30 UTC
Simple patch, adds 2 sysprefs for xslts.
Marking as Passed QA.
Comment 4 Paul Poulain 2012-08-01 15:53:52 UTC
Do we agree that there will be a follow-up to implement this feature ? I've pushed the patch, and hope not needing to revert it later !
Comment 5 Jared Camins-Esakov 2012-08-01 15:56:29 UTC
(In reply to comment #4)
> Do we agree that there will be a follow-up to implement this feature ? I've
> pushed the patch, and hope not needing to revert it later !

Even if it is not used right away, this is certainly useful to have, and I see no reason to plan to revert it.