Project

General

Profile

Actions

Feature #89457

closed

Add possibility to mark as error specific external URLs / domains or patterns

Added by Lina Wolf over 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Linkvalidator
Target version:
-
Start date:
2019-10-18
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
easy
Sprint Focus:

Description

There are several use cases where it is necessary to find all links to a certain domain or url pattern. Use cases include:

- Find all External Links that are indeed pointing to the site itself in order to replace them for internal links

- Find all External Links pointing to a former installation on an external system preparing to shut down.

- Find links to certain sites you do not want to link for legal reasons (some media doesn't want to be deeplinked) or because they have been found to have unreliable information or the domain has been sold or such


Related issues 1 (1 open0 closed)

Related to TYPO3 Core - Feature #85127: linkvalidator: Add possibility to exclude specific external URLs / domains or patternsNew2018-05-31

Actions
Actions #1

Updated by Lina Wolf over 4 years ago

  • Related to Feature #85127: linkvalidator: Add possibility to exclude specific external URLs / domains or patterns added
Actions #2

Updated by Sybille Peters over 3 years ago

  • Status changed from New to Needs Feedback

I think this is a good suggestion and a useful improvement. However, I am currently working on the opposite: make it possible to ignore some specific link targets. In our site it is much more a problem, that some URLs are always reported as broken even though they are not. A lot of this are pages with login, but there are also other reasons.

I had 3 ideas how to solve that and am currently working on a solution where you have an "ignore button": https://review.typo3.org/c/Packages/TYPO3.CMS/+/66608

Any feedback on that would be welcome.

I also had the idea to just add the to-be-ignored URLs to the TSconfig but this is more flexible - it can be made available to editors or not, it could be imported from a file etc.

You requirement could be done in a similar way. But I am currently not sure, how it should be implemented. Maybe you can think about this some more and comment how you would implement it.

Currently, I see at least 2 fields that would be required:

- URL (link target), this can be external, but also a page, file, email etc.
- a reason
- And for each reason, text to display as error - ideally make it possible to translate. We could also provide some standard texts which could be used, e.g. "Do not link to this URL for legal reasons", "Use page links to link to site", "The site will be shut down" etc.

The reason could just be a code and we have some standard texts for that. Alternatively we might make it possible to add your own custom reasons.


I have some more comments:

- for some cases, I would prefer to catch these kind of errors early, e.g. when the URL is entered (though - of course you will still need to find existing cases), e.g. in the Link Browser or on RTE transformations

Find all External Links that are indeed pointing to the site itself in order to replace them for internal links

I think this is a very common usecase. We also have this a lot (editors use external URLs to link to pages on the site). Georg Ringer recently created an extension, which takes care of this partly, but not for rich text fields. We are currently looking into the possibility of funding a solution which will do that - either in the core or as separate extension. If you want to help to fund this, please contact me.

So, I would currently wait how that pans out.

Actions #3

Updated by Sybille Peters over 3 years ago

An alternative way to do this currently, is to override the hook for 'external' => ExternalLinkType with your own class.

Actions #4

Updated by Christian Kuhn about 1 year ago

Hey. I hope it's ok to close here due to lack of further feedback. Feel free to re-open or add a fresh issue in case this should still be handled.

Actions #5

Updated by Christian Kuhn about 1 year ago

  • Status changed from Needs Feedback to Closed
Actions

Also available in: Atom PDF