Bug #8317

Parse Error in FLOW3 script with PHP < 5.3

Added by Bastian Waidelich about 5 years ago. Updated almost 5 years ago.

Status:Resolved Start date:2010-09-26
Priority:Should have Due date:
Assigned To:Karsten Dambekalns % Done:

100%

Category:Core
Target version:TYPO3 Flow Base Distribution - 1.0 alpha 12 Estimated time:1.00 hour
PHP Version: Complexity:
Has patch: Affected Flow version:

Description

When calling php Packages\Framework\FLOW3\Scripts\FLOW3.php with PHP 5.2.x I'm getting a parse error.

$className = '\F3\FLOW3\Core\Bootstrap';
$className::defineConstants();

should have been the workaround to prevent the parser from dying with a parse error. But the second line throws
Parse error: parse error in Packages\Framework\FLOW3\Scripts\FLOW3.php on line 38

on my windows machine with PHP 5.2.10-dev

Associated revisions

Revision 07ca210c
Added by Karsten Dambekalns almost 5 years ago

[+BUGFIX] FLOW3 (Core): Fix parse error on PHP < 5.3

Now anyone using PHP < 5.3 gets a proper error message again.

Change-Id: I952037d183a040d62ac16347fcf0c560f6026161
Fixes: #8317

History

#1 Updated by Karsten Dambekalns almost 5 years ago

  • Project changed from 30 to TYPO3.Flow
  • Category deleted (323)

#2 Updated by Karsten Dambekalns almost 5 years ago

  • Category set to Core
  • Status changed from New to Needs Feedback
  • Assigned To set to Karsten Dambekalns
  • Start date changed from 2010-06-17 to 2010-09-26
  • Estimated time set to 1.00

Without that line, does the version check work as intended?

#3 Updated by Christian Müller almost 5 years ago

Without
$className::defineConstants();

the version check works (just tested). Idea would be to move the version check to /Web/index.php. There it works (tested too).

#4 Updated by Karsten Dambekalns almost 5 years ago

  • Status changed from Needs Feedback to Under Review
  • Target version set to 1.0 alpha 12

#5 Updated by Karsten Dambekalns almost 5 years ago

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

Also available in: Atom PDF