Project

General

Profile

Actions

Bug #79052

closed

Error with PHP 7.1 when calling TypoScriptFrontendController->getConfigArray

Added by Timo Hund almost 8 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
Start date:
2016-12-20
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.1
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

When i run some test from EXT:solr with PHP 7.1 i get the following error:

TypeError: Argument 2 passed to TYPO3\CMS\Core\Utility\ArrayUtility::mergeRecursiveWithOverrule() must be of the type array, string given

I think the reason is, that TypoScriptFrontendController->config is still an empty string, when the part i the code is called, and the second argument was expected to be an array.

if (is_array($this->tmpl->setup['config.'])) {
   ArrayUtility::mergeRecursiveWithOverrule($this->tmpl->setup['config.'], $this->config['config']);
   $this->config['config'] = $this->tmpl->setup['config.'];
}

Actions #1

Updated by Timo Hund almost 8 years ago

  • Subject changed from PHP 7.1 TypoScriptFrontendController->config should be initialized with empty array? to Error with PHP 7.1 when callikng TypoScriptFrontendController->getConfigArray
  • Description updated (diff)
  • Target version set to 8 LTS
  • PHP Version set to 7.1
Actions #2

Updated by Stephan Großberndt almost 8 years ago

  • Description updated (diff)
Actions #3

Updated by Timo Hund almost 8 years ago

With php 7.0 and below everything is fine.

Actions #4

Updated by Timo Hund almost 8 years ago

  • Subject changed from Error with PHP 7.1 when callikng TypoScriptFrontendController->getConfigArray to Error with PHP 7.1 when calling TypoScriptFrontendController->getConfigArray
Actions #5

Updated by Markus Kasten almost 8 years ago

Can confirm. Looks like a PHP 7.1 bug to me.

PHP 7:
@php > $config = '';
php > $config['config'] = [];
php > print_r($config);
Array
(
[config] => Array
(
)

)@

PHP 7.1:
php > $config = '';
php > $config['config'] = [];
php > print_r($config);
A

Actions #6

Updated by Sascha Egerer almost 8 years ago

Fix is coming...

Actions #7

Updated by Gerrit Code Review almost 8 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/51302

Actions #8

Updated by Gerrit Code Review almost 8 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/51302

Actions #9

Updated by Gerrit Code Review almost 8 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51302

Actions #10

Updated by Gerrit Code Review almost 8 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51302

Actions #11

Updated by Sascha Egerer almost 8 years ago

Related PHP "Bug" (It's not a bug, it's a feature)
https://bugs.php.net/bug.php?id=73081

Actions #12

Updated by Gerrit Code Review almost 8 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51302

Actions #13

Updated by Gerrit Code Review almost 8 years ago

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

Actions #14

Updated by Gerrit Code Review almost 8 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/51371

Actions #15

Updated by Sascha Egerer almost 8 years ago

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

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF