Bug #56409

Interface incompatibility - PackageManager

Added by Tymoteusz Motylewski over 7 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2014-02-28
Due date:
% Done:

100%

Estimated time:
PHP Version:
Has patch:
No
Complexity:

Description

PackageManagerInterface contains following method declaration:
public function initialize(\TYPO3\Flow\Core\Bootstrap $bootstrap);

however in the PackageManager (which implements \TYPO3\Flow\Package\PackageManagerInterface)
it is implemented like this:
public function initialize(\TYPO3\Flow\Core\Bootstrap $bootstrap, $packagesBasePath = FLOW_PATH_PACKAGES, $packageStatesPathAndFilename = '') {

this is wrong and will fatal on some versions of php, see http://php.net/manual/en/language.oop5.interfaces.php
"The class implementing the interface must use the exact same method signatures as are defined in the interface. Not doing so will result in a fatal error."

It also fatals when running unit tests on hhvm (of CMS)


Related issues

Related to TYPO3.Flow - Bug #54549: PackageManager::createPackage is incompatible to PackageManagerInterfaceNew2013-12-21

Actions
Related to TYPO3 Core - Bug #56660: Interface incompatibility - PackageManager initialize() - backport from FlowClosed2014-03-07

Actions

Also available in: Atom PDF