Feature #59502

EXT:linkvalidator hooks

Added by Vitoandre D'Oria almost 6 years ago. Updated 12 days ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Linkvalidator
Target version:
-
Start date:
2014-06-11
Due date:
% Done:

0%

PHP Version:
5.3
Tags:
Complexity:
easy
Sprint Focus:

Description

Hello,

we are still sitting on 4.5 but the following things should apply to 6.2 and above:
Please consider adding the following hooks:

1.)
Additional Mail Markers
Story:
We would like to load a signature with an additional marker. On 4.5 XCLASS'ing doesn't seem to work although a XCLASS definition exist it doesn't load the classes at all places.

2.)
The existing hook $GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['linkvalidator']['checkLinks'] isn't helpful imho.
You need to rewrite everything if you want to support TV - FCE's for example. But in fact 80-90% of the existing code could be reused if you would add a hook at the end of analyzeRecord() that would pass all values. Other than that you would only have to change the visibility to public for analyseLinks() and analyseTypoLinks(). With that its possible to use the existing validation code for flexforms and probably many more.


Related issues

Related to TYPO3 Core - Epic #88027: Properly handle Hooks/Signals and Events In Progress 2019-03-28
Related to TYPO3 Core - Feature #29342: More detailed Mail from scheduler task New 2011-08-30

History

#1 Updated by Riccardo De Contardi over 4 years ago

  • Category set to Linkvalidator

#2 Updated by Mathias Schreiber over 4 years ago

  • Target version deleted (next-patchlevel)

#3 Updated by Alexander Opitz about 2 years ago

For point 2.) Yes, that's true, this is really annoying, I needed to XCLass the LinkValidator to make this functions available in public.

As you are speaking of TV - FCEs, I put this into an extension. See https://extensions.typo3.org/extension/ppi_tvplus_linkvalidator/

If this extension is fine for you, I'd like to close this issue and create 2 dedicated issues for your point 1 and 2 (as they are independent and every problem should have his own issue number to resolve too).

#4 Updated by Vitoandre D'Oria about 2 years ago

Thanks for the reply. We are currently locked on 6.2 with the reported system. I can't really test your Extension but from description assume it would be sufficient enough :-)

#5 Updated by Benni Mack about 1 year ago

  • Related to Epic #88027: Properly handle Hooks/Signals and Events added

#6 Updated by Georg Ringer 12 days ago

  • Related to Feature #29342: More detailed Mail from scheduler task added

#7 Updated by Georg Ringer 12 days ago

closing this in favor of #29342 which is about having more information in the mails

#8 Updated by Georg Ringer 12 days ago

  • Status changed from New to Closed

Also available in: Atom PDF