Task #45499
closed\TYPO3\CMS\Utility\StringUtility is not used in core - consider deprecation
100%
Description
Hi.
If my patch for Issue #45404 is accepted, the class \TYPO3\CMS\Utility\StringUtility will not be used in the core or any system extension anymore. Also it seems not that mature in its current state.
Please consider deprecation of that whole class (and its tests, of course).
Updated by Ingo Renner almost 12 years ago
I agree with the issue mentioned in #45404 but I think we still should have a string utility class. Instead of deprecating the class I suggest moving other string related methods here.
Updated by Jost Baron almost 12 years ago
That would be even better, GeneralUtility really needs fewer functions in it.
I'm not sure if deprecation of the current "isLastPartOfStr" would still be a good idea: It is not used in the core (don't know about extensions), and thus is dead code. On the other hand, that means throwing away quite a bit of work (look at the unit tests).
Updated by Gerrit Code Review over 10 years ago
- Status changed from New to Under Review
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30611
Updated by Gerrit Code Review over 10 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30611
Updated by Gerrit Code Review over 10 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30611
Updated by Markus Klein over 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 5110012201ff2a51fc099d3de9d0053bed6f08b6.
Updated by Riccardo De Contardi over 7 years ago
- Status changed from Resolved to Closed