Bug #48052

Using Namespace in t3lib_TCEforms_Suggest breaks PHP 5.2 support

Added by Martin Borer about 8 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Backend API
Target version:
Start date:
2013-05-08
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

Using function:
\TYPO3\CMS\Core\Utility\GeneralUtility::trimExplode

in class.t3lib_tceforms_suggest.php

breaks PHP 5.2 Support.

Solution:
use t3lib_div::trimExplode


Related issues

Related to TYPO3 Core - Bug #21588: TCA type suggest not working in record sys_action type "edit records"Closed2009-11-17

Actions
#1

Updated by Christian Kuhn about 8 years ago

Confirmed. Thanks! Will be taken care of in 4.5.27. Special thanks for this report before this was released!

#3

Updated by Thorsten Kahler about 8 years ago

  • Category set to 1394
  • Status changed from New to Accepted
  • Target version set to 4.5.26
Grepping for TYPO3\ in the current sources of version 4.7 revealed a few other usages of namespaces:
  • Code
    • class Tx_Fluid_Tests_Unit_View_TemplateViewTest
    • class Tx_Fluid_Tests_Unit_ViewHelpers_Form_AbstractFormFieldViewHelperTest
  • PHPDoc
    • class tx_form_System_Postprocessor_Mail
    • class t3lib_stdGraphicTest

I expect to find the same (or even more) issues in 4.5 too!

#4

Updated by Martin Borer about 8 years ago

Typo3 4.7 requires PHP 5.3, using Namspaces in this version is ok.

As far as I can see, there are no other places in Typo3 4.5 where Namespaces are used.

#5

Updated by Gerrit Code Review about 8 years ago

  • Status changed from Accepted to Under Review

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

#6

Updated by Gerrit Code Review about 8 years ago

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

#7

Updated by Nicole Cordes about 8 years ago

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

Updated by Ernesto Baschny almost 8 years ago

  • Is Regression set to Yes
#9

Updated by Ernesto Baschny almost 8 years ago

  • Category changed from 1394 to Backend API
#10

Updated by Riccardo De Contardi over 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF