---- Reported by jwagner@ptfs.com 2009-09-15 14:08:07 ---- I'd like to get some feedback as to whether people think this would be useful. One of my sites asked for a search link from Koha to their SFX OpenURL resolver. (They're using the COinS / OpenURL now, but that apparently requires a browser plugin -- they want a permanent link available to all users.) They want to do this for both ISSN and ISBN because they manage ebooks through SFX as well. I succeeded in modifying the MARC21slim2OPACDetail.xsl file to wrap the ISSN in the syntax to make a call to the SFX server. I'm wondering if (a) people would find this useful as a general patch, and (b) how to set it up to be relatively generic. I'm thinking of two sysprefs (one each for ISBN and ISSN) where the site could fill in the URL of the OpenURL plus the search syntax as a prefix. For example: http://my.sfx.server/?issn= Then, if there is a value in the syspref, pass it into XSLT and make the ISSN/ISBN a link with that syntax. I haven't worked with other OpenURL systems besides SFX. Is this kind of syntax generic enough? Do other systems require any kind of postfix (text after the ISSN/ISBN) as well as a prefix? Ideas? Comments? --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:13 UTC --- This bug was previously known as _bug_ 3634 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3634 Unknown operating system Windows 2000. Setting to default OS "All". Actual time not defined. Setting to 0.0 The original reporter of this bug does not have an account here. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. Previous reporter was jwagner@ptfs.com. The original assignee of this bug does not have an account here. Reassigning to the default assignee for the component, chris@bigballofwax.co.nz. Previous assignee was jwagner@ptfs.com.
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.