Currently the links to edit the source(s) of a broken link are shown on the report in the format "Edit node @nid" and "Edit comment @cid". This isn't particularly user friendly however I assume this was done due to performance. I don't think a call to node_load_multiple() and comment_load_multiple() for each link should cause major issues.

Patch to follow.

CommentFileSizeAuthor
#1 linkchecker-report-links-2403179-1.patch1.33 KBmsmithcti
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

msmithcti’s picture

msmithcti CreditAttribution: msmithcti commented
FileSize
linkchecker-report-links-2403179-1.patch1.33 KB

This patch loads any nodes and comments before printing the links to them, allowing the inclusion of the title which should help users to know exactly what the link allows them to edit.

msmithcti’s picture

msmithcti CreditAttribution: msmithcti commented
Status: Active » Needs review
hass’s picture

hass CreditAttribution: hass commented

That was mostly done for table column size reasons. With title the table will be extremly large and do not fit on a screen.

RumpledElf’s picture

RumpledElf CreditAttribution: RumpledElf commented

Just chiming in, we had a client ask why the broken link report doesn't show which pages have broken links.

The node/id format is *really* not user friendly. Could it be an option to use full titles vs node IDs?

mmikitka’s picture

mmikitka CreditAttribution: mmikitka commented

Related to this issue, I just submitted a patch for linkchecker reports by type: https://www.drupal.org/node/2822413