Bug #40003

FLOW3 not working on PHP 5.3.16

Added by Karsten Dambekalns almost 3 years ago. Updated almost 3 years ago.

Status:Resolved Start date:2012-08-20
Priority:Must have Due date:
Assigned To:Karsten Dambekalns % Done:

100%

Category:Reflection
Target version:TYPO3 Flow Base Distribution - 1.1
PHP Version:5.3 Complexity:
Has patch:No Affected Flow version:(any)

Description

The fix for https://bugs.php.net/62715 landed in PHP 5.3.16, and because of https://bugs.php.net/62874 reflection fails (in Production context) with a pretty unhelpful exception:

Fatal error: Uncaught exception … with message 'Parameter is not optional' in …

Associated revisions

Revision 2dbfd8fe
Added by Karsten Dambekalns almost 3 years ago

[BUGFIX] Fix reflection on PHP 5.3.16

The fix for https://bugs.php.net/62715 landed in PHP 5.3.16, and because of
https://bugs.php.net/62874 reflection fails (in Production context) with a
pretty unhelpful exception:

Fatal error: Uncaught exception … with message 'Parameter is not optional' in …

This change adds an additional check with isOptional() to the relevant code
part.

Change-Id: Ib7e763340f51fcd2d4f41291a4b8b80c33459d6b
Fixes: #40003
Releases: 1.1, 1.2

Revision 28c82ae2
Added by Karsten Dambekalns almost 3 years ago

[BUGFIX] Fix reflection on PHP 5.3.16

The fix for https://bugs.php.net/62715 landed in PHP 5.3.16, and because of
https://bugs.php.net/62874 reflection fails (in Production context) with a
pretty unhelpful exception:

Fatal error: Uncaught exception … with message 'Parameter is not optional' in …

This change adds an additional check with isOptional() to the relevant code
part.

Change-Id: Ib7e763340f51fcd2d4f41291a4b8b80c33459d6b
Fixes: #40003
Releases: 1.1, 1.2

History

#1 Updated by Karsten Dambekalns almost 3 years ago

  • Target version set to 1.1

#2 Updated by Gerrit Code Review almost 3 years ago

  • Status changed from Accepted to Under Review

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

#3 Updated by Karsten Dambekalns almost 3 years ago

Ok, the bug will be fixed again in 5.3.17. Sigh.

#4 Updated by Karsten Dambekalns almost 3 years ago

Discussed with Robert and we'll simply go for the additional isOptional() check, because (although we could) we never supported optional parameters followed by non-optional parameters.

#5 Updated by Gerrit Code Review almost 3 years ago

Patch set 1 for branch FLOW3-1.1 has been pushed to the review server.
It is available at http://review.typo3.org/13983

#6 Updated by Karsten Dambekalns almost 3 years ago

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

#7 Updated by Thomas Deinhamer almost 3 years ago

I'm experiencing the same problem on Extbase in TYPO 4.5.19;
will this fix be merged into Extbase as well?

Also available in: Atom PDF