Project

General

Profile

Actions

Bug #62807

closed

Merge array functions from GeneralUtility to ArrayUtility

Added by Frederic Gaus over 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Category:
Code Cleanup
Target version:
Start date:
2014-11-10
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Move the following functions to ArrayUtilities:

\TYPO3\CMS\Core\Utility\GeneralUtility::inArray
\TYPO3\CMS\Core\Utility\GeneralUtility::removeArrayEntryByValue
\TYPO3\CMS\Core\Utility\GeneralUtility::keepItemsInArray
\TYPO3\CMS\Core\Utility\GeneralUtility::remapArrayKeys
\TYPO3\CMS\Core\Utility\GeneralUtility::array_merge
\TYPO3\CMS\Core\Utility\GeneralUtility::arrayDiffAssocRecursive
\TYPO3\CMS\Core\Utility\GeneralUtility::naturalKeySortRecursive

it has to be discussed if this functions should be moved to ArrayUtilities too or if they should better be moved to StringUtilities:
\TYPO3\CMS\Core\Utility\GeneralUtility::trimExplode
\TYPO3\CMS\Core\Utility\GeneralUtility::revExplode
\TYPO3\CMS\Core\Utility\GeneralUtility::intExplode


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Feature #64596: Deprecate \TYPO3\CMS\Core\Utility\GeneralUtility::array_mergeClosedGöran Bodenschatz2015-01-29

Actions
Actions #1

Updated by Mathias Schreiber over 9 years ago

  • Target version changed from 7.0 to 7.1 (Cleanup)
Actions #2

Updated by Anja Leichsenring about 9 years ago

  • Sprint Focus set to On Location Sprint
Actions #3

Updated by Göran Bodenschatz about 9 years ago

  • Assignee set to Göran Bodenschatz
Actions #4

Updated by Göran Bodenschatz about 9 years ago

::array_merge stays in GeneralUtility as it'll get deprecated

::trimExplode, ::revExplode, and ::intExplode should stay in GeneralUtility

Actions #5

Updated by Göran Bodenschatz about 9 years ago

  • Status changed from New to Accepted
Actions #6

Updated by Gerrit Code Review about 9 years ago

  • Status changed from Accepted 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/36371

Actions #7

Updated by Gerrit Code Review about 9 years 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/36371

Actions #8

Updated by Gerrit Code Review about 9 years 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/36371

Actions #9

Updated by Gerrit Code Review about 9 years 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/36371

Actions #10

Updated by Gerrit Code Review about 9 years 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/36371

Actions #11

Updated by Stephan Großberndt about 9 years ago

  • Status changed from Under Review to In Progress
Actions #12

Updated by Gerrit Code Review about 9 years ago

  • Status changed from In Progress to Under Review

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/36371

Actions #13

Updated by Gerrit Code Review about 9 years ago

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

Actions #14

Updated by Göran Bodenschatz about 9 years ago

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

Updated by Anja Leichsenring over 8 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions #16

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF