Bug #41213

Tree view in admin panel is broken on PHP 5.4

Added by Philipp Gampe over 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Backend API
Target version:
-
Start date:
2012-09-21
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
3.5
PHP Version:
5.4
Tags:
Complexity:
medium
Is Regression:
Sprint Focus:

Description

The code is actually broken since the initial commit, but "worked" in PHP <5.4.

The problematic part is a wrong array access in the timetrack->printTSlog() function.

It took me quite a while to debug this.

To verify:
  • open the admin panel
  • open TypoScript
  • check
    • Tree display
    • Track content rendering
    • Force TS Rendering
  • see no tree with PHP 5.4 (Fedora)

I also have the patch for TYPO3 4.x ready.


Related issues

Has duplicate TYPO3 Core - Bug #39302: PHP Warning: Invalid argument supplied for foreach() in t3lib/class.t3lib_timetrack.php line 397, 404, 436Rejected2012-07-26

Actions
#1

Updated by Gerrit Code Review over 8 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14880

#2

Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/14920

#3

Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/14920

#4

Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/14923

#5

Updated by Philipp Gampe over 8 years ago

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

Updated by Gerrit Code Review over 8 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/14924

#7

Updated by Philipp Gampe over 8 years ago

  • Status changed from Under Review to Resolved
#8

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF