Summary: | RIS format export not exporting URI field | ||
---|---|---|---|
Product: | Koha | Reporter: | Chris Cormack <chris> |
Component: | OPAC | Assignee: | Chris Cormack <chris> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | minor | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | gmcharlt, 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: | ||
Circulation function: | |||
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
Created attachment 3186 [details] [review] Proposed Patch 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. 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 Created attachment 5232 [details] [review] signed-off original patch Created attachment 5233 [details] [review] follow-up patch that addresses the multiple 856 issue raised during review of the original patch Signed off on original patch and submitted follow-up patch that addresses the issue raised during QA. Both patches are easy to read, and followup patch adds a simple loop to multiple URLs. Marking as Passed QA Patches still apply for me on current master - perhaps wrong status was set? Chris, Katrin and I can both get this to apply on current master. Could you describe the error your encountering in more detail? Yeah my bad, sorry, pushing now There have been no further reports of problems so I am marking this bug resolved. |