Bug #51486
closed
InstallSysExtsUpdate::installExtensions() incompatible AbstractUpdate::installExtensions()
0%
Description
PHP Runtime Notice: Declaration of TYPO3\CMS\Install\CoreUpdates\InstallSysExtsUpdate::installExtensions() should be compatible with TYPO3\CMS\Install\Updates\AbstractUpdate::installExtensions($extensionKeys) in D:\wamp\www\typo3_src-6.1.3\typo3\sysext\install\Classes\CoreUpdates\InstallSysExtsUpdate.php line 38
Amount of parameters is different.
Updated by Philipp Gampe over 10 years ago
- Status changed from New to Accepted
I think the overridding of the function was not intentional. Thus the function should berenamed to e.g. fetchAndInstallExtensions(...) in InstallSysExtsUpdate.
Could you push a patch to gerrit? http://wiki.typo3.org/CWT
Updated by Tomasz Krawczyk over 10 years ago
How whold you call InstallSysExtsUpdate::markWizardAsDone() ?
PHP Runtime Notice: Declaration of TYPO3\CMS\Install\CoreUpdates\InstallSysExtsUpdate::markWizardAsDone() should be compatible with TYPO3\CMS\Install\Updates\AbstractUpdate::markWizardAsDone($confValue = 1) in D:\wamp\www\typo3_src-6.1.3\typo3\sysext\install\Classes\CoreUpdates\InstallSysExtsUpdate.php line 38
I think PHP 5.3 allows on such overriding while PHP 5.4 does not.
PS. There are C++ programmers among us. ;-)
Updated by Tomasz Krawczyk over 10 years ago
The erroe message from my previous post shows after change InstallSysExtsUpdate::installExtensions to InstallSysExtsUpdate::fetchAndInstallExtensions.
Updated by Mathias Schreiber over 9 years ago
- Status changed from Accepted to Needs Feedback
- Assignee set to Mathias Schreiber
- Is Regression set to No
Is this still the case on 6.2?
Updated by Mathias Schreiber over 9 years ago
- Status changed from Needs Feedback to Closed
alrighty. Thanks for the feedback