Task #57499

Improve deprecation message of GeneralUtitlity::array_merge_recursive_overrule()

Added by Georg Ringer over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-04-01
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:- Spent time: -
Target version:next-patchlevel
TYPO3 Version:6.2 Complexity:
PHP Version: Sprint Focus:

Description

The deprecation message should show more clearly that the code can't be just updated by changing class + method name as there is no return value anymore. This is new in the core and is a quite big risk!


Related issues

related to Core - Task #54251: GeneralUtility::array_merge_recursive_overrule has bad pe... Resolved 2013-12-06

Associated revisions

Revision ae220dcd
Added by Georg Ringer over 1 year ago

[TASK] Improve deprecation msg of GeneralUtility method

The deprecation message of GeneralUtility::array_merge_recursive_overrule
should show more clearly that the code can't be just updated by
changing class + method name as there is no return value anymore.

This is new in the core and is a quite big risk!

Resolves: #57499
Releases: 6.2
Change-Id: I109a0c3edbc241708e7b362a249b6d0e4a79978e
Reviewed-on: https://review.typo3.org/29035
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

History

#1 Updated by Gerrit Code Review over 1 year 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 https://review.typo3.org/29035

#2 Updated by Gerrit Code Review over 1 year ago

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

#3 Updated by Georg Ringer over 1 year ago

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

#4 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch master of project Teams/Security/TYPO3v4-Core has been pushed to the review server.
It is available at https://review.typo3.org/29229

#5 Updated by Helmut Hummel over 1 year ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF