Project

General

Profile

Actions

Bug #82304

closed

ConfigurationManager::writeLocalConfiguration breaks log writer configuration

Added by Thorsten Kahler about 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
System/Bootstrap/Configuration
Start date:
2017-09-05
Due date:
% Done:

100%

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

Description

Method \TYPO3\CMS\Core\Configuration\ConfigurationManager::writeLocalConfiguration() cleans up $TYPO3_CONF_VARS before writing to typo3conf/LocalConfiguration.php. It uses the cleanup method ArrayUtility::renumberKeysToAvoidLeapsIfKeysAreAllNumeric() which rewrites the indices of numerically indexed arrays.

This cleanup breaks $TYPO3_CONF_VARS['LOG']['writerConfiguration'] because the configuration uses log level constants which represent integers (see class \TYPO3\CMS\Core\Log\LogLevel) as indices.

Either exceptions for those cleanups must be configurable or (better) this optimization attempt should be dropped at all.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #44938: LocalConfiguration extListArray adds keys if extension is uninstalledClosedJan-Erik Revsbech2013-01-29

Actions
Actions #1

Updated by Oliver Hader about 7 years ago

Does this only occur in master or in stable releases as well?

Actions #2

Updated by Thorsten Kahler about 7 years ago

The optimization attempt was already introduced in version 6.1 and backported to 6.0, see change c1d929d5.

Actions #3

Updated by Thorsten Kahler about 7 years ago

  • Related to Bug #44938: LocalConfiguration extListArray adds keys if extension is uninstalled added
Actions #4

Updated by Gerrit Code Review over 6 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/55936

Actions #5

Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55943

Actions #6

Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55944

Actions #7

Updated by Helmut Hummel over 6 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF