From 2ab9d22d4c24a82971895d0b0a00101b1fc47680 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Chris=20M=C3=BCller?= <2566282+brotkrueml@users.noreply.github.com> Date: Thu, 19 Dec 2024 14:22:18 +0100 Subject: [PATCH] Avoid PHP 8.4 deprecations Fix the following deprecation messages: PHP Deprecated: YoastSeoForTypo3\YoastSeo\Backend\PageLayoutHeader::render(): Implicitly marking parameter $params as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/vendor/yoast-seo-for-typo3/yoast_seo/Classes/Backend/PageLayoutHeader.php on line 27 PHP Deprecated: YoastSeoForTypo3\YoastSeo\Controller\AbstractBackendController::returnResponse(): Implicitly marking parameter $moduleTemplate as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/vendor/yoast-seo-for-typo3/yoast_seo/Classes/Controller/AbstractBackendController.php on line 25 PHP Deprecated: YoastSeoForTypo3\YoastSeo\EventListener\RecordCanonicalListener::__construct(): Implicitly marking parameter $recordService as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/vendor/yoast-seo-for-typo3/yoast_seo/Classes/EventListener/RecordCanonicalListener.php on line 16 PHP Deprecated: YoastSeoForTypo3\YoastSeo\MetaTag\Generator\AbstractGenerator::__construct(): Implicitly marking parameter $managerRegistry as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/vendor/yoast-seo-for-typo3/yoast_seo/Classes/MetaTag/Generator/AbstractGenerator.php on line 19 PHP Deprecated: YoastSeoForTypo3\YoastSeo\PageTitle\RecordPageTitleProvider::__construct(): Implicitly marking parameter $recordService as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/vendor/yoast-seo-for-typo3/yoast_seo/Classes/PageTitle/RecordPageTitleProvider.php on line 16 PHP Deprecated: YoastSeoForTypo3\YoastSeo\Utility\JavascriptUtility::loadJavascript(): Implicitly marking parameter $pageRenderer as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/vendor/yoast-seo-for-typo3/yoast_seo/Classes/Utility/JavascriptUtility.php on line 13 --- Classes/Backend/PageLayoutHeader.php | 2 +- Classes/Controller/AbstractBackendController.php | 2 +- Classes/EventListener/RecordCanonicalListener.php | 2 +- Classes/MetaTag/Generator/AbstractGenerator.php | 2 +- Classes/PageTitle/RecordPageTitleProvider.php | 2 +- Classes/Utility/JavascriptUtility.php | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/Classes/Backend/PageLayoutHeader.php b/Classes/Backend/PageLayoutHeader.php index 29e32dbd..1a2f3677 100644 --- a/Classes/Backend/PageLayoutHeader.php +++ b/Classes/Backend/PageLayoutHeader.php @@ -24,7 +24,7 @@ public function __construct( /** * @param array|null $params */ - public function render(array $params = null, PageLayoutController|ModuleTemplate|null $parentObj = null): string + public function render(?array $params = null, PageLayoutController|ModuleTemplate|null $parentObj = null): string { $languageId = $this->getLanguageId(); $pageId = (int)$_GET['id']; diff --git a/Classes/Controller/AbstractBackendController.php b/Classes/Controller/AbstractBackendController.php index 9799f72b..19f134a2 100644 --- a/Classes/Controller/AbstractBackendController.php +++ b/Classes/Controller/AbstractBackendController.php @@ -22,7 +22,7 @@ abstract class AbstractBackendController extends ActionController /** * @param array $data */ - protected function returnResponse(string $template, array $data = [], ModuleTemplate $moduleTemplate = null): ResponseInterface + protected function returnResponse(string $template, array $data = [], ?ModuleTemplate $moduleTemplate = null): ResponseInterface { $data['layout'] = GeneralUtility::makeInstance(Typo3Version::class) ->getMajorVersion() < 13 ? 'Default' : 'Module'; diff --git a/Classes/EventListener/RecordCanonicalListener.php b/Classes/EventListener/RecordCanonicalListener.php index 0d1162a4..769da538 100644 --- a/Classes/EventListener/RecordCanonicalListener.php +++ b/Classes/EventListener/RecordCanonicalListener.php @@ -13,7 +13,7 @@ class RecordCanonicalListener { protected RecordService $recordService; - public function __construct(RecordService $recordService = null) + public function __construct(?RecordService $recordService = null) { if ($recordService === null) { $recordService = GeneralUtility::makeInstance(RecordService::class); diff --git a/Classes/MetaTag/Generator/AbstractGenerator.php b/Classes/MetaTag/Generator/AbstractGenerator.php index f9bc4e28..6574755f 100644 --- a/Classes/MetaTag/Generator/AbstractGenerator.php +++ b/Classes/MetaTag/Generator/AbstractGenerator.php @@ -16,7 +16,7 @@ abstract class AbstractGenerator implements GeneratorInterface { protected MetaTagManagerRegistry $managerRegistry; - public function __construct(MetaTagManagerRegistry $managerRegistry = null) + public function __construct(?MetaTagManagerRegistry $managerRegistry = null) { if ($managerRegistry === null) { $managerRegistry = GeneralUtility::makeInstance(MetaTagManagerRegistry::class); diff --git a/Classes/PageTitle/RecordPageTitleProvider.php b/Classes/PageTitle/RecordPageTitleProvider.php index 14d9e878..9505dfcd 100644 --- a/Classes/PageTitle/RecordPageTitleProvider.php +++ b/Classes/PageTitle/RecordPageTitleProvider.php @@ -13,7 +13,7 @@ class RecordPageTitleProvider extends AbstractPageTitleProvider { protected RecordService $recordService; - public function __construct(RecordService $recordService = null) + public function __construct(?RecordService $recordService = null) { if ($recordService === null) { $recordService = GeneralUtility::makeInstance(RecordService::class); diff --git a/Classes/Utility/JavascriptUtility.php b/Classes/Utility/JavascriptUtility.php index 327f9bb4..0fb45822 100644 --- a/Classes/Utility/JavascriptUtility.php +++ b/Classes/Utility/JavascriptUtility.php @@ -10,7 +10,7 @@ class JavascriptUtility { - public static function loadJavascript(PageRenderer $pageRenderer = null): void + public static function loadJavascript(?PageRenderer $pageRenderer = null): void { if ($pageRenderer === null) { $pageRenderer = GeneralUtility::makeInstance(PageRenderer::class);