External links added through link module, in the following format https://patents.google.com/?q=test+search+param are being treated as http://localhost/test+search+param due to call to drupal_parse_url() in _linkchecker_parse_fields().

So these links are showing up as 404.

Comments

rwaery.11 created an issue. See original summary.

rwaery.11’s picture

Issue summary: View changes
rwaery.11’s picture

rwaery.11’s picture

Status: Active » Needs review

Status: Needs review » Needs work
rwaery.11’s picture

Status: Needs work » Needs review
hass’s picture

Version: 7.x-1.3 » 7.x-1.x-dev
Issue tags: -external links

Status: Needs review » Needs work
hass’s picture

Code style issues and no documentation why this is needed.

joseph.olstad’s picture

this might also affect D8

eiriksm’s picture

This does not affect D8. Tried to add the link in the issue summary, and it was checked and gave a 200.

The patch still needs work though.

It would be great to clean up the coding standards and document the change somewhat better, as suggested in #9.

quadrexdev’s picture

Status: Needs work » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

If the issue remains relevant for D10+ versions, merge requests with proposed solutions for a new module version (D10+) are welcome in a new follow-up issue.

Thanks!

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.