Task #64109

Deprecate softRefParser_GL

Added by Stephan Großberndt 7 months ago. Updated 7 months ago.

Status:Resolved Start date:2015-01-04
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:- Spent time: -
Target version:-
TYPO3 Version:7 Complexity:easy
PHP Version: Sprint Focus:

Description

typo3/sysext/backend/Classes/Utility/BackendUtility.php defines a general hook "softRefParser_GL" to add a general softRefParser which parses every SoftReference regardless of its type. The "softRefParser_GL"-hook is undocumented and used neither in core nor in any known extension.

http://docs.typo3.org/typo3cms/TCAReference/AdditionalFeatures/SoftReferences/Index.html

There are several types of SoftReferences defined:

There is a default configuration in typo3/sysext/core/Configuration/DefaultConfiguration.php mapping "softRefParser" for several types to TYPO3\CMS\Core\Database\SoftReferenceIndex. Also rtehtmlarea adds a parser for "rtehtmlarea_images".

Removing it would mean a "softRefParser_GL" hook in an extension would have to be replaced with multiple "softRefParser" hooks for each type the parser can handle instead of having one "softRefParser_GL" hook declaring "I want to parse all!". You cannot know how every possible softref has to be parsed, so registering for every type the parser knows to handle is perfectly valid.

Therefore softRefParser_GL should be deprecated in CMS 7 and removed in CMS 8.


Related issues

related to Core - Bug #63674: High memory consumption in BackendUtility->explodeSoftRef... Resolved 2014-12-08

Associated revisions

Revision 10459346
Added by Stephan Großberndt 7 months ago

[!!!][TASK] Deprecate global hook softRefParser_GL

The hook ['SC_OPTIONS']['GLOBAL']['softRefParser_GL'] has been
deprecated. It was a hook to add a general softRefParser which parsed
every SoftReference regardless of its type. The "softRefParser_GL"-hook
was undocumented and used neither in core nor in any known extension.

Resolves: #64109
Releases: master
Change-Id: I1a69097f50e8002e559bd89d39b9ed72dfa18e03
Reviewed-on: http://review.typo3.org/35773
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#1 Updated by Gerrit Code Review 7 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35773

#2 Updated by Gerrit Code Review 7 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35773

#3 Updated by Gerrit Code Review 7 months ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35773

#4 Updated by Gerrit Code Review 7 months ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35773

#5 Updated by Gerrit Code Review 7 months ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35773

#6 Updated by Gerrit Code Review 7 months ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35773

#7 Updated by Stephan Großberndt 7 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF