Project

General

Profile

Actions

Feature #59502

closed

EXT:linkvalidator hooks

Added by Vitoandre D'Oria almost 10 years ago. Updated about 4 years ago.

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

0%

Estimated time:
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 2 (0 open2 closed)

Related to TYPO3 Core - Epic #88027: Properly handle Hooks/Signals and EventsClosedBenni Mack2022-04-22

Actions
Related to TYPO3 Core - Feature #29342: More detailed Mail from scheduler taskClosed2011-08-30

Actions
Actions #1

Updated by Riccardo De Contardi over 8 years ago

  • Category set to Linkvalidator
Actions #2

Updated by Mathias Schreiber over 8 years ago

  • Target version deleted (next-patchlevel)
Actions #3

Updated by Alexander Opitz about 6 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).

Actions #4

Updated by Vitoandre D'Oria about 6 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 :-)

Actions #5

Updated by Benni Mack about 5 years ago

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

Updated by Georg Ringer about 4 years ago

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

Updated by Georg Ringer about 4 years ago

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

Actions #8

Updated by Georg Ringer about 4 years ago

  • Status changed from New to Closed
Actions

Also available in: Atom PDF