Task #45499

\TYPO3\CMS\Utility\StringUtility is not used in core - consider deprecation

Added by Jost Baron over 8 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Code Cleanup
Target version:
Start date:
2013-02-15
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
easy
Sprint Focus:

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).


Related issues

Related to TYPO3 Core - Feature #45404: em should not export invisible foldersClosed2013-02-12

Actions
Related to TYPO3 Core - Task #59132: Move GenralUtility::isFirstPartOfStr() to StringUtility::isFirstPartOfStr()Closed2014-05-27

Actions
#1

Updated by Ingo Renner over 8 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.

#2

Updated by Jost Baron over 8 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).

#3

Updated by Gerrit Code Review about 7 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

#4

Updated by Gerrit Code Review almost 7 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

#5

Updated by Gerrit Code Review almost 7 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

#6

Updated by Markus Klein almost 7 years ago

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

Updated by Riccardo De Contardi over 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF