| Summary: | Add links to related authorities for UNIMARC | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Julian Maurice <julian.maurice> |
| Component: | Staff interface | Assignee: | Julian Maurice <julian.maurice> |
| Status: | CLOSED FIXED | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | acovaliov, gmcharlt, jonathan.druart, martin.renvoize, nick, sandboxes |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | Small patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 19550: Add links to related authorities for UNIMARC
Bug 19550: Add links to related authorities for UNIMARC Bug 19550: Add links to related authorities for UNIMARC Bug 19550: (RM follow-up) Add filters Bug 19550: (QA follow-up on RM follow-up) Add missing [% USE %] |
||
|
Description
Julian Maurice
2017-10-31 12:37:32 UTC
Created attachment 68856 [details] [review] Bug 19550: Add links to related authorities for UNIMARC Links exist in search results for MARC21 but not for UNIMARC. This patch fixes that. Test plan: 1. Create an authority with a field 550 that links to another authority e.g. 550 $a Foo $9 42 2. Reindex this authority 3. Search for this authority 4. See that you now have a link "Foo" to authorities/detail.pl?authid=42 in the summary Patch tested with a sandbox, by delaye <stephane.delaye@biblibre.com> Created attachment 72871 [details] [review] Bug 19550: Add links to related authorities for UNIMARC Links exist in search results for MARC21 but not for UNIMARC. This patch fixes that. Test plan: 1. Create an authority with a field 550 that links to another authority e.g. 550 $a Foo $9 42 2. Reindex this authority 3. Search for this authority 4. See that you now have a link "Foo" to authorities/detail.pl?authid=42 in the summary Signed-off-by: delaye <stephane.delaye@biblibre.com> Failing a bit getting a UNIMARC db up right now. Created attachment 78089 [details] [review] Bug 19550: Add links to related authorities for UNIMARC Links exist in search results for MARC21 but not for UNIMARC. This patch fixes that. Test plan: 1. Create an authority with a field 550 that links to another authority e.g. 550 $a Foo $9 42 2. Reindex this authority 3. Search for this authority 4. See that you now have a link "Foo" to authorities/detail.pl?authid=42 in the summary Signed-off-by: delaye <stephane.delaye@biblibre.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Ended up testing on a sandbox, but worked well there :) I think the template variables need the $raw filter, they are set within the templaet itself so should be safe, but we need filters to keep tests happy Created attachment 78490 [details] [review] Bug 19550: (RM follow-up) Add filters I can't get a UNIMARC db working right now, can you verify the followup works Julian? If so I will push Created attachment 78530 [details] [review] Bug 19550: (QA follow-up on RM follow-up) Add missing [% USE %] (In reply to Nick Clemens from comment #9) > I can't get a UNIMARC db working right now, can you verify the followup > works Julian? If so I will push With my additional follow-up, it works ;) Awesome work all! Pushed to master for 18.11 Enhancement, will not be backported t 18.05.x series. Nick, I think the filter should be 'url', not raw |