Project

General

Profile

Actions

Bug #20675

open

Broken Link Checker no longer sending email notifications

Added by Raffi Khatchadourian 4 months ago. Updated 4 months ago.

Status:
New
Priority name:
Normal
Assignee:
-
Category name:
-
Target version:
-
Start date:
2024-07-31
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

I haven't seen any emails from the Broken Link Checker since early 2023. Today, I checked out the page, and there are two new broken links. Yet, I don't have any emails about them.

Side note: I looked at an old email from Broken Link Checker. The link to view the broken links no longer works. The link is https://khatchad.commons.gc.cuny.edu/wp-admin/tools.php?page=view-broken-links. I get, "Sorry, you are not allowed to access this page," when I go there.

Actions #1

Updated by Raymond Hoh 4 months ago

Hi Raffi,

It looks like someone already reported this issue here: https://wordpress.org/support/topic/email-notification-action-skipped-for-multisite-subsites/

The developers say they will address this in a future release. If this isn't important, can it wait until then?

The reporter posted a workaround, but I think the workaround touches too many parts of the Broken Link Checker codebase. The alternative is to make a core change to the plugin and removing three lines until the plugin makes the change themselves.

Side note: I looked at an old email from Broken Link Checker. The link to view the broken links no longer works. The link is https://khatchad.commons.gc.cuny.edu/wp-admin/tools.php?page=view-broken-links. I get, "Sorry, you are not allowed to access this page," when I go there.

The link to view broken links should be: https://khatchad.commons.gc.cuny.edu/wp-admin/admin.php?page=blc_local&filter_id=broken

Actions #2

Updated by Raffi Khatchadourian 4 months ago

Hi Raymond,

Sorry for the delay:

Raymond Hoh wrote in #note-1:>

It looks like someone already reported this issue here: https://wordpress.org/support/topic/email-notification-action-skipped-for-multisite-subsites/

Good find!

The developers say they will address this in a future release. If this isn't important, can it wait until then?

Yes. I don't think it's been working for quite a while.

Side note: I looked at an old email from Broken Link Checker. The link to view the broken links no longer works. The link is https://khatchad.commons.gc.cuny.edu/wp-admin/tools.php?page=view-broken-links. I get, "Sorry, you are not allowed to access this page," when I go there.

The link to view broken links should be: https://khatchad.commons.gc.cuny.edu/wp-admin/admin.php?page=blc_local&filter_id=broken

That's great, but the link in the email is still incorrect. Should we file a bug on the plug-in? AFAIK, it was correct at some point. Thanks!

Actions #3

Updated by Boone Gorges 4 months ago

Looking through the codebase, it appears that broken-link-checker changed its internal link structure around version 2.2.x. Specifically, the callback responsible for the view-broken-links routing was disabled around that version. See https://plugins.trac.wordpress.org/browser/broken-link-checker/trunk/app/options/settings/class-controller.php?rev=2927805&annotate=blame&marks=53#L45. We updated from 2.1.0 to 2.2.2 on Sep 25, 2023. So it's possible that your old notification email came before that date, but the link no longer functioned properly after that date. If so, this is not great backward compatibility on behalf of the plugin, but at the same time it's only relevant for emails that are quite old, and so may not be worth the trouble of a fix.

Actions #4

Updated by Raffi Khatchadourian 4 months ago

Ah, I see. So, in the new email, it may be a different link.

Actions

Also available in: Atom PDF