Created attachment 104604 [details] Screenshot showing the problem In authority records there is the option to designate a subfield as a URL (like we do for 856$u in the default MARC framework). When you view the details of such a record the URL is output to the screen as HTML markup. This is because the script is sending hard-coded HTML to the template and the template outputs is as escaped. Rather than use the $raw filter to make the link work, we should remove the script-generated HTML and do it in the template.
Created attachment 104605 [details] [review] Bug 25428: Correctly encode link if authority subfield is a URL This patch modifies the script and template for showing the details of an authority record so that if the authority subfield is configured to be a link the link is correctly output in the template. To test you must have an authority type configured with at least one subfield designated as a URL: - Go to Administration -> Authority types - Select "MARC structure" from the actions menu for the authority type you want to modify. - Select "Subfields" from the actions menu for the tag containing the subfield you want to modify (for instance 856). - Edit the subfield you want to display as a link (e.g. subfield u). - In the subfield configuration form under "Advanced constraints" check the "Is a URL" checkbox and save your changes. - Apply the patch and go to Authorities in the staff interface. - Search for or create an authority record which contains URL information in the right subfield (for instance 856$u). - View the details for the record. Using the 856 example, the "8" tab should contain an active link pointing to the correct URL.
Created attachment 104967 [details] [review] Bug 25428: Correctly encode link if authority subfield is a URL This patch modifies the script and template for showing the details of an authority record so that if the authority subfield is configured to be a link the link is correctly output in the template. To test you must have an authority type configured with at least one subfield designated as a URL: - Go to Administration -> Authority types - Select "MARC structure" from the actions menu for the authority type you want to modify. - Select "Subfields" from the actions menu for the tag containing the subfield you want to modify (for instance 856). - Edit the subfield you want to display as a link (e.g. subfield u). - In the subfield configuration form under "Advanced constraints" check the "Is a URL" checkbox and save your changes. - Apply the patch and go to Authorities in the staff interface. - Search for or create an authority record which contains URL information in the right subfield (for instance 856$u). - View the details for the record. Using the 856 example, the "8" tab should contain an active link pointing to the correct URL. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Work as described, no errors
Created attachment 105200 [details] [review] Bug 25428: Correctly encode link if authority subfield is a URL This patch modifies the script and template for showing the details of an authority record so that if the authority subfield is configured to be a link the link is correctly output in the template. To test you must have an authority type configured with at least one subfield designated as a URL: - Go to Administration -> Authority types - Select "MARC structure" from the actions menu for the authority type you want to modify. - Select "Subfields" from the actions menu for the tag containing the subfield you want to modify (for instance 856). - Edit the subfield you want to display as a link (e.g. subfield u). - In the subfield configuration form under "Advanced constraints" check the "Is a URL" checkbox and save your changes. - Apply the patch and go to Authorities in the staff interface. - Search for or create an authority record which contains URL information in the right subfield (for instance 856$u). - View the details for the record. Using the 856 example, the "8" tab should contain an active link pointing to the correct URL. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Work as described, no errors Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Nice work everyone! Pushed to master for 20.05
backported to 19.11.x for 19.11.07
Backported to 19.05.x branch for 19.05.12