Project

General

Profile

Actions

Bug #15930

closed

Importing/installing skin_grey_2 in 4.0 RC2 reports Invalid argument supplied for foreach() .. class.em_index.php

Added by Palahala about 18 years ago. Updated over 17 years ago.

Status:
Closed
Priority:
Should have
Category:
Extension Manager
Target version:
-
Start date:
2006-03-28
Due date:
% Done:

0%

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

Description

As the extension itself has not changed since I successfully loaded it into 4.0 RC1, and as I did not run into this error in RC1, I am reporting this for the Extension Manager.

Importing skin_grey_2 version 0.0.1 (according to http://typo3.org/extensions/repository/search/skin_grey_2/0.0.1/details/ that is created 25.03.2006 - 18:47) from TER into 4.0 RC2 gives me

Warning: Invalid argument supplied for foreach() in [...]/typo3_src-4.0rc2/typo3/mod/tools/em/class.em_index.php on line 3956

Next, installing it yields the same error. It seems that the end result is fine though (the skin looks good). Even uninstalling gives the same error.

I uninstalled and removed the extension, cleared temp_CACHED files in typo3conf and imported and installed again -- with the same error.

The error occurs in:

3950 function checkDependencies($extKey, $conf, $instExtInfo) {
:
3956 foreach($conf['constraints']['depends'] as $depK => $depV) {

It might very well only apply to extensions that have no dependencies, as the generated ext_emconf.php for skin_grey_2 shows (partially copied)

'title' => 'Gray/orange skin for TYPO3 4.0',
'constraints' => array(
'' => array(
'' => '',
),
),

The "constraints" item looks different from, for example, t3skin:

'title' => 'TYPO3 skin',
'constraints' => array(
'depends' => array(
'php' => '4.3.0-5.9.0',
'typo3' => '3.9.0-4.0',
),
'conflicts' => array(
),
'suggests' => array(
),
),

Likewise, tipafriend (which has no depencies either) shows:

'title' => 'Tip-A-Friend',
'constraints' => array(
'depends' => array(
),
'conflicts' => array(
),
'suggests' => array(
),
),

So maybe the extension is wrong, but again: I did not get this error in 4.0 RC1 and maybe more (erroneous) extensions will now suddenly show the same error. To avoid this, an additional check for a value for

$conf['constraints']['depends']

might avoid existing extensions from breaking?

(issue imported from #M3033)


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #15936: Gremlin: The running TYPO3 version (4.0-CVS) is higher than allowed (0.0.2)ClosedKarsten Dambekalns2006-03-29

Actions
Actions #1

Updated by Marc Bastian Heinrichs about 18 years ago

confirm
If dependencies field of the EXT is empty, $conf['constraints']['depends'] is empty and the foreach produces the warning.

Actions #2

Updated by Karsten Dambekalns about 18 years ago

If the dependencies are empty and there are no constraints, the constrainst get set to an empty array, the warning will not be trigerred.

I can reproduce this error, but the emconf as contained in the download already has a broken constraints array. I added a stricter check to fixEMCONF() so it'll fix such a broken constraints array, too. In addition I changed arrayToCode() to skip empty keys (this where the broken constraints come from).

As soon as the constraints entry is an array and has one of the keys depends, conflicts, suggests the corresponding fix/check will be skipped, as it is assumed the thing is correct!

Anyway, it works here now without any warnings.

Actions

Also available in: Atom PDF