Bug #40113

Task #40095: Move core to namespaces

Superfluous brackets in functions after namespaces move

Added by Steffen Müller almost 9 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
-
Target version:
-
Start date:
2012-08-24
Due date:
% Done:

100%

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

Description

The following syntax appears in various parts of the core:

$logger->warning(((((('Instantiation of LogWriter "' .
$logWriterClassName) . '" failed for logger ') . $logger->getName()) . '
(') . $e->getMessage()) . ')');

see: http://forge.typo3.org/projects/typo3v4-core/repository/revisions/master/entry/typo3/sysext/core/Classes/Log/LogManager.php#L146


Related issues

Related to TYPO3 Core - Task #40777: Superfluous brackets in functions after namespaces moveClosedArvid Jakobsson2012-09-09

Actions
Related to TYPO3 Core - Task #40779: Superfluous brackets in the version-sysext after namespaces moveClosedArvid Jakobsson2012-09-09

Actions
Related to TYPO3 Core - Task #40774: Superfluous brackets in functions after namespaces moveClosedArvid Jakobsson2012-09-09

Actions
Related to TYPO3 Core - Task #40878: Superfluous brackets in sysext backend after namespaces moveClosedArvid Jakobsson2012-09-12

Actions
Related to TYPO3 Core - Task #40880: Superfluous brackets in sysext "core" after namespaces moveClosedArvid Jakobsson2012-09-13

Actions
Related to TYPO3 Core - Task #40962: Superfluous brackets in sysexts "aboutmodules", "adodb", "belog" and "beuser" after namespaces moveClosedArvid Jakobsson2012-09-16

Actions
#1

Updated by Christian Kuhn almost 9 years ago

  • Parent task set to #40095
#2

Updated by Christian Kuhn almost 9 years ago

Confirmed. The new parenthesis are semantically identical with the old code, so it shouldn't break. We want to create a cleanup in this area later on.

#3

Updated by Steffen Gebert almost 9 years ago

  • Status changed from New to Accepted
#4

Updated by Arvid Jakobsson almost 9 years ago

Hey people, I'm working on this one. I've got lex-pass setup to automatically fix this issue. I'm gonna start by fixing each of the sysexts, and will then move on to the core.

#5

Updated by Wouter Wolters over 8 years ago

This is completed. Please close:)

#6

Updated by Thomas Maroschik over 8 years ago

  • Status changed from Accepted to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF