Project

General

Profile

Actions

Feature #59502

closed

EXT:linkvalidator hooks

Added by Vitoandre D'Oria over 10 years ago. Updated over 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 about 9 years ago

  • Category set to Linkvalidator
Actions #2

Updated by Mathias Schreiber about 9 years ago

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

Updated by Alexander Opitz over 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 over 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 over 5 years ago

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

Updated by Georg Ringer over 4 years ago

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

Updated by Georg Ringer over 4 years ago

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

Actions #8

Updated by Georg Ringer over 4 years ago

  • Status changed from New to Closed
Actions

Also available in: Atom PDF