Bug #92181
closedTYPO3\CMS\Core\Configuration\ExtensionConfiguration->set paremter; anotation for third parameter wrong
100%
Description
Is
/** * @param string $extension Extension name * @param string $path Configuration path to set - eg. "featureCategory/coolThingIsEnabled" * @param null $value The value. If null, unset the path * @internal */ public function set(string $extension, string $path = '', $value = null): void {
should be
/** * @param string $extension Extension name * @param string $path Configuration path to set - eg. "featureCategory/coolThingIsEnabled" * @param string|null $value The value. If null, unset the path * @internal */ public function set(string $extension, string $path = '', ?string $value = null): void {
Triggers PHP Stan violation:
Parameter #3 $value of method TYPO3\CMS\Core\Configuration\ExtensionConfiguration::set() expects null, string given.
Updated by Timo Poppinga about 4 years ago
- Subject changed from TYPO3\CMS\Core\Configuration->set paremter; anoteded for third parameter wrong to TYPO3\CMS\Core\Configuration->set paremter; anotation for third parameter wrong
Updated by Timo Poppinga about 4 years ago
- Subject changed from TYPO3\CMS\Core\Configuration->set paremter; anotation for third parameter wrong to TYPO3\CMS\Core\Configuration\ExtensionConfiguration->set paremter; anotation for third parameter wrong
Updated by Gerrit Code Review about 4 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65541
Updated by Gerrit Code Review about 4 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65541
Updated by Gerrit Code Review about 4 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65541
Updated by Gerrit Code Review about 4 years ago
Patch set 1 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65567
Updated by Timo Poppinga about 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 1e109fd2e282d19bc8157ed445c80f80fb9aa906.