Problem/Motivation

As a user, I can run the broken link report for the entire website from the Managed Link Status Report.

Proposed resolution

Add a new page to allow the user to run the broken link report on demand

Remaining tasks

Review.
Commit.
Rejoice.

User interface changes

New page

When there is no link entity

When there is one link entity pending.

When there is already a job running.

CommentFileSizeAuthor
#2 3093121-2.patch7.77 KBjibran
screenshot-127.0.0.1-2019.11.08-13_38_07.png86.02 KBjibran
screenshot-127.0.0.1-2019.11.08-13_44_01.png81.87 KBjibran
screenshot-127.0.0.1-2019.11.08-13_40_05.png82.06 KBjibran
screenshot-127.0.0.1-2019.11.08-13_44_24.png78.2 KBjibran

Issue fork linkychecker-3093121

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

jibran created an issue. See original summary.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Active » Needs review
StatusFileSize
new 3093121-2.patch7.77 KB

Here is the patch.

dpi’s picture

dpi
Location Perth, Australia
commented

This pretty much just does what the cron job does. It doesnt force links to be checked despite their last checked time, and it doesnt actually check links in the same request, only creates queue items.

acbramley’s picture

acbramley commented
Status: Needs review » Reviewed & tested by the community

We've been running this patch for >3 years now without issue, let's get it in :)

acbramley’s picture

acbramley commented
Version: 8.x-1.x-dev » 2.2.x-dev

Still applies to 2.2.x

dpi opened merge request !4

dpi’s picture

dpi
Location Perth, Australia
commented

Deferring #3 to future bug/feature issue.

  • dpi committed 39087deb on 2.2.x 
    Issue #3093121 by jibran, dpi: Add ability to run broken link report on...
dpi’s picture

dpi
Location Perth, Australia
commented
Status: Reviewed & tested by the community » Fixed

Merged!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.