Project

General

Profile

Actions

Task #42293

closed

Please check for array, to avoid strict mode warnings

Added by Michael Schramm over 11 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
Workspaces
Target version:
Start date:
2012-10-23
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
5.3
Tags:
Complexity:
Sprint Focus:

Description

Dear Team,

please add a check on array ->
&& !is_array ($content)

in typo3\sysext\workspaces\Classes\Service\GridData.php
tx_Workspaces_Service_GridData::getDataArrayFromCache ()

on line 238
...
if ($content != FALSE && !is_array ($content) ) {
...

This avoids protocol warnings for strict mode


Related issues 1 (0 open1 closed)

Is duplicate of TYPO3 Core - Bug #34117: Usage of caching framework broken in TYPO3 4.5ClosedOliver Hader2012-02-20

Actions
Actions #1

Updated by Michael Schramm over 11 years ago

Warning examples

03:00:53 . [5] ! [5_0] Core: Error handler (BE): PHP Warning: unserialize() expects parameter 1 to be string, array given in ...typo3/sysext/workspaces/Classes/Service/GridData.php line 243

02:19:15 . . ! . Core: Error handler (BE): PHP Warning: unserialize() expects parameter 1 to be string, array given in .../typo3/sysext/workspaces/Classes/Service/GridData.php line 239

Actions #2

Updated by Oliver Hader over 11 years ago

  • Target version set to 4.5.23
  • PHP Version set to 5.3

I prefer to analyse the origin of the warning and have that fixed instead of not showing it.
see issue #34117 for further details and how the behavior can be fixed - a patch is available as well

This behaviour occurs in TYPO3 4.5 only.

Actions #3

Updated by Oliver Hader over 11 years ago

  • Status changed from New to Closed
Actions #4

Updated by Michael Stucki over 10 years ago

  • Category changed from Labels / Error Messages to Workspaces
Actions #5

Updated by Michael Stucki over 10 years ago

  • Project changed from 624 to TYPO3 Core
  • Category changed from Workspaces to Workspaces
Actions

Also available in: Atom PDF