Problem/Motivation

Skip pattern is trying to match missing parts of the path resulting in a PHP warning.

Steps to reproduce

1. Create an exempt path like `/*/page/*`
2. visit a page with links to check
3. verify warning

Proposed resolution

Update elseif on line 875 of LinkChecker.php to be elseif (!array_key_exists($index, $path_parts) || $pattern_part !== $path_parts[$index])

Issue fork node_link_report-3575859

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

brian.seek created an issue. See original summary.

brian.seek’s picture

brian.seek commented
Issue summary: View changes

brian.seek opened merge request !14

skyriter’s picture

skyriter commented

I have tested patch #3 successfully.

skyriter’s picture

skyriter commented
Status: Needs review » Reviewed & tested by the community
swirt’s picture

swirt
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thank you brian.seek and skyriter for your work on this. I will do a release shortly.

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.

swirt’s picture

swirt
he/him
Primary language English
commented

Just released with #3576257: node_link_report 8.x-1.25

Status: Fixed » Closed (fixed)

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