"Print this publication" link returns a 404

Created on 8 September 2024, 2 months ago
Updated 19 September 2024, 2 months ago

Problem/Motivation

The "Print this publication" link in MultipageActionsBlock has a few issues.

  • Use of href="javascript:void(0);" - javascript is not an allowed protocol as so is removed in Xss::attributes - this then renders as href="void(0);"
  • Link has js-multipage-print-page - but there is no JS in the module providing print functionality or using this selector

Steps to reproduce

  • Enable module and place block on page.
  • Click the "Print this publication" link

Proposed resolution

  • Replace javascript:void(0); with #
  • Add rel=nofollow to print publication link
  • Include JS to trigger the print behaviour
πŸ› Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

πŸ‡³πŸ‡ΏNew Zealand ericgsmith

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024