Project

General

Profile

Actions

Bug #23851

closed

PHP warning in System reports extension

Added by Rudi Meyer about 14 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2010-10-27
Due date:
% Done:

0%

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

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 #1

Updated by Steffen Gebert about 14 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 #2

Updated by Rudi Meyer about 14 years ago

Hi Steffen,

I have attached a patch with my fix.

Thanks for a quick response!
Rudi

Actions #3

Updated by Steffen Gebert about 14 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 #4

Updated by Rudi Meyer about 14 years ago

I have send the patch to the Core Team list

Actions #5

Updated by Christian Kuhn about 14 years ago

Committed to:

  • trunk 9223
  • 4.4 9224
  • 4.3 9225
Actions #6

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF