| 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 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 | 
| 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 |