Task #64109
Deprecate softRefParser_GL
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
Associated revisions
[!!!][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 <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <klein.t3@reelworx.at>
Tested-by: Markus Klein <klein.t3@reelworx.at>
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
Applied in changeset 104593467782de3d2df2008fcb22d1a1cb0f71d4.