Bug #60101: Cleanup - Stupid array assignment - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #60101

closed

Cleanup - Stupid array assignment

Added by Sascha Egerer over 10 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Sascha Egerer

Category:

Code Cleanup

Target version:

next-patchlevel

Start date:

2014-07-04

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

6.2

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

In FormEngine.php there are two cases where the following assignment is done:

$setAll = array();
$setAll[] = 'foo';
$setAll[] .= 'bar';

The .= does not make sense because it also adds a new array entry.

Actions

Copy link

Updated by Gerrit Code Review over 10 years ago

Status changed from New 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/31321

Actions

Copy link

Updated by Gerrit Code Review over 10 years 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/31321

Actions

Copy link