Currently, URLs are not showing up in the OPAC basket (more detail view). This is because the template tries to get them with the key "MARCurlS" when the actual key is "MARCURLS". The template should also use MARCurl.url instead of MARCurl.part, so that users can see the actual URL rather than the "Materials specified".
Created attachment 38605 [details] [review] Bug 12253: Fix MARCUrls in basket At the OPAC and the intranet, the urls don't be displayed anymore. The variables used in the templates are not the good ones (MARCurlS vs MARCURLS). Test plan: 1/ On the intranet side, add some urls to some records 2/ Add these records to the basket 3/ Add records without urls defined 4/ Go on the basket view, click "more details" 5/ You should see the urls displayed 6/ Repeat steps 4-5 at the OPAC
Created attachment 38634 [details] [review] Bug 12253: Fix MARCUrls in basket At the OPAC and the intranet, the urls don't be displayed anymore. The variables used in the templates are not the good ones (MARCurlS vs MARCURLS). Test plan: 1/ On the intranet side, add some urls to some records 2/ Add these records to the basket 3/ Add records without urls defined 4/ Go on the basket view, click "more details" 5/ You should see the urls displayed 6/ Repeat steps 4-5 at the OPAC Signed-off-by: Mark Tompsett <mtompset@hotmail.com> NOTE: 856$u displays now in intranet and OPAC. :)
Created attachment 38691 [details] [review] [PASSED QA] Bug 12253: Fix MARCUrls in basket At the OPAC and the intranet, the urls don't be displayed anymore. The variables used in the templates are not the good ones (MARCurlS vs MARCURLS). Test plan: 1/ On the intranet side, add some urls to some records 2/ Add these records to the basket 3/ Add records without urls defined 4/ Go on the basket view, click "more details" 5/ You should see the urls displayed 6/ Repeat steps 4-5 at the OPAC Signed-off-by: Mark Tompsett <mtompset@hotmail.com> NOTE: 856$u displays now in intranet and OPAC. :) Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Patches pushed to master. Thanks Jonathan!
Pushed to 3.18.x will be in 3.18.7