Summary: | Adding OpenURL link to XSLT ISSN/ISBN | ||
---|---|---|---|
Product: | Koha | Reporter: | Jane Wagner <jwagner> |
Component: | OPAC | Assignee: | Jane Wagner <jwagner> |
Status: | CLOSED WONTFIX | QA Contact: | |
Severity: | enhancement | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | jcamins |
Version: | Main | ||
Hardware: | PC | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
patch file against current head
Screen shot of link resolver display revised patch file |
Description
Chris Cormack
2010-05-21 01:13:07 UTC
I worked up a patch which I think will allow for any link resolver. There are two system preferences: OPAClinkresolver (for whether or not to use it) and OPAClinkresolverURL (to enter the URL and parameters for the link resolver). If the feature is turned on, and there is an ISSN in the MARC record, a "Check Full Text Availability" link will show in the upper right box on the details page, along with the Place Hold, Print, etc. links. I designed the layout so that both the generic image and the wording can be changed by stylesheet and jquery settings. This would allow a site to use, say, the SFX logo and a local wording like Check Our Databases instead. Note: this could have been accomplished by adding an entry into the OPACSearchForTitleIn setup. I chose to do it separately for two reasons -- to make it more prominent on the page (instead of hidden in the More Searches pulldown), and so that it would only display if the feature was turned on and there was an ISSN in the record. Created attachment 2435 [details] [review] patch file against current head Created attachment 2436 [details]
Screen shot of link resolver display
Created attachment 2591 [details] [review] revised patch file Patch file revised to add support for 3.2 sysprefs editor. This patch does not apply. Also, it renames a file, which seems strange. Here are the messages I got: Commit Body is: -------------------------- Bug 3634 REVISED Add support for link resolver such as SFX Creates two sysprefs to enable link resolver and enter a URL for the link resolver server. Link will appear in righthand box on the details page, along with Place Hold, Print, etc. -------------------------- Apply? [y]es/[n]o/[e]dit/[v]iew patch/[a]ccept all a Applying: Bug 3634 REVISED Add support for link resolver such as SFX Using index info to reconstruct a base tree... <stdin>:125: trailing whitespace. - "</code>example: <code>http://mysfxserver.com/sfxlcl3/?sfx.ignore_date_threshold=1&issn=</code>" <stdin>:32: new blank line at EOF. + <stdin>:43: new blank line at EOF. + <stdin>:74: new blank line at EOF. + warning: 4 lines add whitespace errors. Falling back to patching base and 3-way merge... Renaming koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced-content.pref => koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced_content.pref Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/enhanced_content.pref Auto-merging admin/systempreferences.pl Auto-merging installer/data/mysql/de-DE/mandatory/sysprefs.sql CONFLICT (content): Merge conflict in installer/data/mysql/de-DE/mandatory/sysprefs.sql Auto-merging installer/data/mysql/en/mandatory/sysprefs.sql CONFLICT (content): Merge conflict in installer/data/mysql/en/mandatory/sysprefs.sql Auto-merging installer/data/mysql/fr-FR/1-Obligatoire/unimarc_standard_systemprefs.sql CONFLICT (content): Merge conflict in installer/data/mysql/fr-FR/1-Obligatoire/unimarc_standard_systemprefs.sql Auto-merging installer/data/mysql/pl-PL/mandatory/sysprefs.sql CONFLICT (content): Merge conflict in installer/data/mysql/pl-PL/mandatory/sysprefs.sql Auto-merging installer/data/mysql/ru-RU/mandatory/system_preferences_full_optimal_for_install_only.sql CONFLICT (content): Merge conflict in installer/data/mysql/ru-RU/mandatory/system_preferences_full_optimal_for_install_only.sql Auto-merging installer/data/mysql/updatedatabase.pl Auto-merging koha-tmpl/opac-tmpl/prog/en/css/opac.css CONFLICT (content): Merge conflict in koha-tmpl/opac-tmpl/prog/en/css/opac.css Auto-merging koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tmpl Auto-merging opac/opac-detail.pl Failed to merge in the changes. I don't think this is a sufficiently general solution. A better option would be to add ISSN as a token for More searches, and use that in the site-appropriate manner for OpenURL resolvers. |