Bug #23851: PHP warning in System reports extension - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #23851

closed

PHP warning in System reports extension

Added by Rudi Meyer over 13 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Christian Kuhn

Category:

Target version:

Start date:

2010-10-27

Due date:

% Done:

Estimated time:

TYPO3 Version:

4.4

PHP Version:

5.2

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

With a fresh install of Typo3 4.4.4 the "System reports" extension generates the following error:

Core: Error handler (BE): PHP Warning: array_merge() [<a href='function.array-merge'>function.array-merge</a>]: Argument #1 is not an array in /mnt/web1-data-primary/www/lib/typo3_src-4.4.4/typo3/sysext/reports/ext_tables.php line 19

Missing check for existing array?

OS: Debian GNU/Linux 5.0
(issue imported from #M16140)

Files

Download all files

bug_16140.diff (519 Bytes) bug_16140.diff		Administrator Admin, 2010-10-27 12:33
16140_v2.diff (628 Bytes) 16140_v2.diff		Administrator Admin, 2010-10-27 13:08
16140_03.diff (682 Bytes) 16140_03.diff		Administrator Admin, 2010-10-29 16:36

Actions

Copy link

Updated by Steffen Gebert over 13 years ago

Hi Rudi,

seems so. Are you able to create a patch and only merge, if $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['reports']['tx_reports']['status'] is already an array?

Thanks!
Steffen

Actions

Copy link

Updated by Rudi Meyer over 13 years ago

Hi Steffen,

I have attached a patch with my fix.

Thanks for a quick response!
Rudi

Actions

Copy link

Updated by Steffen Gebert over 13 years ago

Hi Rudi,

thanks for your patch!

I've attached an updated one, as old and new source were mixed up and we usually create our patches against SVN root (typo3_src/).

Would you mind sending this to the Core Team list, to get this committed to the TYPO3 sources? Please choose as branches 4-4 and trunk (if it applies to 4-3, then this one, too).

You can find a how-to here: http://typo3blogger.de/ab-in-den-core/ (I assume you're German ;)).

If you have any questions, don't hesistate to ask!

Thanks for your contribution!
Steffen

Actions

Copy link