Bug #40113

Task #40095: Move core to namespaces

Superfluous brackets in functions after namespaces move

Added by Steffen Müller almost 3 years ago. Updated almost 3 years ago.

Status:Closed Start date:2012-08-24
Priority:Must have Due date:
Assigned To:- % Done:

100%

Category:- Spent time: -
Target version:-
TYPO3 Version:6.0 Is Regression:
PHP Version: Sprint Focus:
Complexity:

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 Core - Task #40774: Superfluous brackets in functions after namespaces move Resolved 2012-09-09
related to Core - Task #40777: Superfluous brackets in functions after namespaces move Resolved 2012-09-09
related to Core - Task #40779: Superfluous brackets in the version-sysext after namespac... Resolved 2012-09-09
related to Core - Task #40878: Superfluous brackets in sysext backend after namespaces move Resolved 2012-09-12
related to Core - Task #40880: Superfluous brackets in sysext "core" after namespaces move Resolved 2012-09-13
related to Core - Task #40962: Superfluous brackets in sysexts "aboutmodules", "adodb", ... Resolved 2012-09-16

History

#1 Updated by Christian Kuhn almost 3 years ago

  • Parent task set to #40095

#2 Updated by Christian Kuhn almost 3 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 3 years ago

  • Status changed from New to Accepted

#4 Updated by Arvid Jakobsson almost 3 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 almost 3 years ago

This is completed. Please close:)

#6 Updated by Thomas Maroschik almost 3 years ago

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

Also available in: Atom PDF