Bug 5662

Summary: Show control number without organization code in OPAC URLs
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: OPACAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: chris, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5663
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 5622    
Bug Blocks:    
Attachments: Patch
[SIGNED-OFF] 5662: Show control number without organization code in OPAC URLs

Description Marcel de Rooy 2011-01-27 13:55:08 UTC
According to LOC guidelines, the control number in $w may be prefixed by an
organization code between parentheses like (NL-AmRIJ)12345. The URL should only
list the record number 12345. (Currently, the complete field is copied into the URL and therefore it is not working.) 

Revised patch 5622 corrected this for the host entry (773) in staff client and included generic XSLT code to resolve the problem. 

This followup patch will build upon the generic code in the Utils.xsl and include calls to that routine at the OPAC side. Another bug report builds upon 5622 for the staff client.
Comment 1 Marcel de Rooy 2011-07-14 08:33:33 UTC Comment hidden (obsolete)
Comment 2 Marcel de Rooy 2011-07-14 08:35:37 UTC
TEST PLAN:

Enable the pref UseControlNumber when testing this patch. And set pref OPACXSLTDetailsDisplay to using XSLT stylesheets.
Test by adding a control number ($w) in abovementioned fields.
Go to opac detail and check if the links work.
Add some orgcode between parentheses or brackets in front of the control number.
Test again in opac detail if the links still work.
Comment 3 Katrin Fischer 2011-07-17 11:09:21 UTC
Created attachment 4642 [details] [review]
[SIGNED-OFF] 5662: Show control number without organization code in OPAC URLs

OPAC detail xsl adjusted at four places: 773 when leader19=c, 775, 780, 785.

Enable the pref UseControlNumber when testing this patch. And set pref OPACXSLTDetailsDisplay to using XSLT stylesheets.
Test by adding a control number ($w) in abovementioned fields.
Go to opac detail and check if the links work.
Add some orgcode between parentheses or brackets in front of the control number.
Test again in opac detail if the links still work.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works with and without orgCode in front of $w now.
No change to current behaviour, but closer to MARC21 standard.
Comment 4 Chris Cormack 2011-08-01 01:42:35 UTC
XSLT changes, pushed, please test
Comment 5 Marcel de Rooy 2011-10-03 07:26:17 UTC
Closing it myself; runs in production with us for months.