Bug 5780

Summary: RIS format export not exporting URI field
Product: Koha Reporter: Chris Cormack <chris>
Component: OPACAssignee: Chris Cormack <chris>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: gmcharlt, katrin.fischer, koha.sekjal
Version: 3.4   
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: 6769    
Bug Blocks:    
Attachments: Proposed Patch
signed-off original patch
follow-up patch that addresses the multiple 856 issue raised during review of the original patch

Description Chris Cormack 2011-02-20 20:58:42 UTC
When exporting a record as RIS, the 856u field is being ignored
Comment 1 Chris Cormack 2011-02-20 21:31:18 UTC Comment hidden (obsolete)
Comment 2 Katrin Fischer 2011-02-22 22:18:21 UTC
Applied patch to master branch.

From OPAC the RIS export included the first URL:
UR  - http://www.loc.gov/catdir/toc/...

In staff the export options for RIS and BibTex are missing.

I tried adding a second URL, but it didn't show up in the RIS export file. Only the first URL was exported.

http://www.refman.com/support/risformat_tags_07.asp 
"UR  - "
Web/URL. There is no practical limit to the length of this field. URL addresses can be entered individually, one per tag or multiple addresses can be entered on one line using a semi-colon as a separator.
Comment 3 MJ Ray (software.coop) 2011-03-05 10:34:33 UTC
This bug is mentioned in:
Bug 5780 - Export the URI as part of a RIS	export http://lists.koha-community.org/pipermail/koha-patches/2011-February/013902.html
Comment 4 Galen Charlton 2011-08-30 19:39:51 UTC
Created attachment 5232 [details] [review]
signed-off original patch
Comment 5 Galen Charlton 2011-08-30 19:40:24 UTC
Created attachment 5233 [details] [review]
follow-up patch that addresses the multiple 856 issue raised during review of the original patch
Comment 6 Galen Charlton 2011-08-30 19:41:48 UTC
Signed off on original patch and submitted follow-up patch that addresses the issue raised during QA.
Comment 7 Ian Walls 2011-09-22 14:59:19 UTC
Both patches are easy to read, and followup patch adds a simple loop to multiple URLs.  Marking as Passed QA
Comment 8 Katrin Fischer 2011-10-13 06:02:12 UTC
Patches still apply for me on current master - perhaps wrong status was set?
Comment 9 Ian Walls 2011-10-13 15:54:52 UTC
Chris,


Katrin and I can both get this to apply on current master.  Could you describe the error your encountering in more detail?
Comment 10 Chris Cormack 2011-10-13 21:39:52 UTC
Yeah my bad, sorry, pushing now
Comment 11 Jared Camins-Esakov 2012-12-31 00:26:19 UTC
There have been no further reports of problems so I am marking this bug resolved.