Bug #54449

Fatal error in configuration module while modifying TYPO3_LOADED_EXT

Added by Maik Hagenbruch about 6 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Backend API
Target version:
-
Start date:
2013-12-16
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
5.4
Tags:
Complexity:
medium
Is Regression:
No
Sprint Focus:

Description

When choosing $TYPO3_LOADED_EXT (Loaded Extensions) in configuration module a fatal error occurs:

#1361915596: The array $GLOBALS['TYPO3_LOADED_EXT'] may not be modified

This happens because the tree method tries to override values in the non-writeable TYPO3_LOADED_EXT array.

\TYPO3\CMS\Lowlevel\View\ConfigurationView : 236

$tree = $arrayBrowser->tree($theVar, '', '');

\TYPO3\CMS\Lowlevel\Utility\ArrayBrowser : 111

if (is_object($arr[$key])) {
  $arr[$key] = (array) $arr[$key];
}

Associated revisions

Revision cbbb4def (diff)
Added by Jigal van Hemert almost 6 years ago

[BUGFIX] Don't modify variables in tree view

The lowlevel array browser tries to modify the array that
is displayed if it's an object.
Now that TYPO3_LOADED_EXT is a simulated array (an object
which implements array access) modifying the data leads to
a fatal error. Instead the iterated value is now modified.

Releases: 6.2
Resolves: #54449
Change-Id: Ib1d3eb5cc76a4180ea0891d88c16191cd16f36e2
Reviewed-on: https://review.typo3.org/27850
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

History

#1 Updated by Thomas Maroschik about 6 years ago

  • Status changed from New to Accepted
  • Complexity set to medium

#2 Updated by Thomas Maroschik about 6 years ago

  • Category set to Backend API

#3 Updated by Gerrit Code Review almost 6 years 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/27850

#4 Updated by Gerrit Code Review almost 6 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/27850

#5 Updated by Jigal van Hemert almost 6 years ago

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

#6 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF