Links exist in search results for MARC21 but not for UNIMARC.
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