I am missing the "own broken links report".
There is a permission 'access own broken links report' stating 'Allows users to access their user specific broken links report.'

This permissions is never used (at least I could not find it besides linkchecker.permissions.yml).
Or did I miss anything?

I think the support was dropped with #2992604: Broken links report (switch to views reports).

Maybe we should provide another display to the broken_links_report view?

CommentFileSizeAuthor
#13 SCR-20260917-jeme.png291.98 KBjoelpittet

Issue fork linkchecker-3334562

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

gngn created an issue. See original summary.

webflo’s picture

webflo commented
Version: 8.x-1.x-dev » 2.1.x-dev

joelpittet made their first commit to this issue’s fork.

joelpittet opened merge request !149

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Active » Needs review

I would like to see this in to get the broken links in the hands of the authors (content editors) that can fix them best.

joelpittet changed the visibility of the branch 2.1.x to hidden.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

Poking to see if anybody can give this test to get RTBC

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Fixed

Thank you @gngn for making the request. I have merged it into 2.1.x-dev and I will merge it into 3.0.x as well.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

codebymikey’s picture

codebymikey commented
Status: Fixed » Needs review

The merge train failed to merge due to a merge conflict, so I think I think this needs revisiting.

Just had a quick test and all works pretty well, just a few questions/review:
1. I think the view probably needs to be flagged to use the Administration theme otherwise the UI can look off on frontend themes.
2. Unsure if the title should be "My broken links" / "Own broken links" or left as "Broken links" since the latter isn't clear unless they look at the URL.
3. Should this page be linked to anywhere - it's not showing up in my tabs or menu when I visit /user/{user}.

codebymikey’s picture

codebymikey commented
Status: Needs review » Needs work
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

@codebymikey thank's for stepping in with a review, I merged late at night (when I have time) and kinda figured one would fail.

I had screenshots, but I think I forgot to post them. The views are optional, so just applying the MR will not import the optional Views config.

I will post the screenshots and consider your title/admin theme feedback before committing again. The goal was to provide parity with missing D7 views display so I will probably keep things close to that as possible in terms of naming.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs work » Needs review
StatusFileSize
new SCR-20260917-jeme.png291.98 KB

report page

Thanks again for the review, @codebymikey. The merge conflict is resolved and I’ve pushed updates for your points:

  • Admin theme: kept the report in the front-end theme, consistent with core’s other user/{user}/* tabs and D7. Sites can still enable the admin theme on the display if desired.
  • Title: now reads “Broken links report for {display name}”. The tab remains “Broken links”.
  • Missing tab: existing installs need drush updb because the view is optional config. On Drupal 11 with Navigation enabled, the link appears under the “⋮” menu instead of classic tabs. Tests now cover tab access.

I also hid the empty Operations column for users without edit permission, including via post-update for existing sites.

Setting back to Needs review. If you test on the same site, please run drush updb first.

  • joelpittet committed 36f6121a on 2.1.x 
    task: #3334562 Missing "own broken links report"

By: gngn
By: webflo
By...
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Fixed

Ok merged for real this time

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.