Project

General

Profile

Actions

Task #45499

closed

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

Added by Jost Baron about 11 years ago. Updated over 6 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 2 (0 open2 closed)

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
Actions #1

Updated by Ingo Renner about 11 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.

Actions #2

Updated by Jost Baron about 11 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).

Actions #3

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

Actions #4

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

Actions #5

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

Actions #6

Updated by Markus Klein almost 10 years ago

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

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF