- Merge request !23Issue #3247823: Internal links set in the link field are not recognized β (Open) created by eiriksm
- π©πͺGermany LAKdev
Since this issue is still relevant in 2.0, I wrote a patch that removes the internal:/ schema when extracting links, i.e. directly in the LinkLinkExtractor Plugin. That way the linkchecker module treats it like any other relative link.
- First commit to issue fork.
- last update
8 months ago 87 pass - πΊπ¦Ukraine quadrexdev Lutsk
Invalid internal links are unrouted ones so I think ->isRouted() check is not what we need, looks like just adding !$url_obj->isExternal() should be enough.
It worked fine on my local using the 9.5.11 core release. I do believe it should be a part of 2.0.x release as well.
- πΊπ¦Ukraine quadrexdev Lutsk
quadrexdev β changed the visibility of the branch 8.x-1.x to hidden.
- last update
8 months ago 87 pass - πΊπ¦Ukraine quadrexdev Lutsk
quadrexdev β changed the visibility of the branch 2.0.x to hidden.
- Status changed to Needs review
8 months ago 2:21pm 10 April 2024 - last update
8 months ago Patch Failed to Apply -
quadrexdev β
committed 6a2d3ff8 on 2.0.x
#3247823 "Internal links set in the link field are not recognized"
-
quadrexdev β
committed 6a2d3ff8 on 2.0.x
- Status changed to Fixed
4 months ago 1:43pm 5 August 2024 - πΊπ¦Ukraine quadrexdev Lutsk
Double-checked, seems to be fine.
Committed on 2.0.x.
Automatically closed - issue fixed for 2 weeks with no activity.