[BUGFIX] fix all Implicitly marked null parameters (PHP 8.4)

Classes/Controller/AbstractFluxController.php

@@ -285,7 +285,7 @@ protected function resolveView(): ViewInterface
     /**
      * @return \Psr\Http\Message\ResponseInterface|Response
      */
-    protected function createHtmlResponse(string $html = null)
+    protected function createHtmlResponse(?string $html = null)
     {
         if (method_exists($this, 'htmlResponse')) {
             return parent::htmlResponse($html);

Classes/Form/Transformation/FormDataTransformer.php

@@ -46,7 +46,7 @@ public function __construct(FlexFormService $flexFormService, DataTransformerReg
      */
     public function convertFlexFormContentToArray(
         string $flexFormContent,
-        Form $form = null,
+        ?Form $form = null,
         ?string $languagePointer = 'lDEF',
         ?string $valuePointer = 'vDEF'
     ): array {

Classes/Integration/Overrides/AbstractChimeraConfigurationManager.php

@@ -24,8 +24,8 @@ abstract class AbstractChimeraConfigurationManager extends ConfigurationManager
 
     public function getConfiguration(
         string $configurationType,
-        string $extensionName = null,
-        string $pluginName = null
+        ?string $extensionName = null,
+        ?string $pluginName = null
     ): array {
         $configurationManager = $this->resolveConfigurationManager();
 

Classes/Integration/Overrides/LocalizationController.php

@@ -24,7 +24,7 @@ class LocalizationController extends \TYPO3\CMS\Backend\Controller\Page\Localiza
      */
     public function getRecordLocalizeSummary(
         ServerRequestInterface $request,
-        ResponseInterface $response = null
+        ?ResponseInterface $response = null
     ): ResponseInterface {
         $params = $request->getQueryParams();
         if (!isset($params['pageId'], $params['destLanguageId'], $params['languageId'])) {

Classes/Integration/PreviewView.php

@@ -45,7 +45,7 @@ class PreviewView extends TemplateView
     protected ConfigurationManagerInterface $configurationManager;
     protected WorkspacesAwareRecordService $workspacesAwareRecordService;
 
-    public function __construct(RenderingContextInterface $context = null)
+    public function __construct(?RenderingContextInterface $context = null)
     {
         parent::__construct($context);
 
@@ -94,7 +94,7 @@ public function getPreview(ProviderInterface $provider, array $row, bool $withou
         )['preview'];
     }
 
-    protected function getPreviewOptions(Form $form = null): array
+    protected function getPreviewOptions(?Form $form = null): array
     {
         if (!is_object($form) || !$form->hasOption(PreviewOption::PREVIEW)) {
             return [
@@ -116,7 +116,7 @@ protected function getOptionToggle(array $options = []): bool
         return (boolean) ($options[PreviewOption::TOGGLE] ?? true);
     }
 
-    protected function renderPreviewSection(ProviderInterface $provider, array $row, Form $form = null): ?string
+    protected function renderPreviewSection(ProviderInterface $provider, array $row, ?Form $form = null): ?string
     {
         $templatePathAndFilename = $provider->getTemplatePathAndFilename($row);
         if (!$templatePathAndFilename) {

Classes/Provider/ProviderResolver.php

@@ -47,7 +47,7 @@ public function resolvePageProvider(array $row): ?ProviderInterface
     public function resolvePrimaryConfigurationProvider(
         ?string $table,
         ?string $fieldName,
-        array $row = null,
+        ?array $row = null,
         ?string $extensionKey = null,
         array $interfaces = [ProviderInterface::class]
     ) {
@@ -66,7 +66,7 @@ public function resolvePrimaryConfigurationProvider(
     public function resolveConfigurationProviders(
         ?string $table,
         ?string $fieldName,
-        array $row = null,
+        ?array $row = null,
         ?string $extensionKey = null,
         array $interfaces = [ProviderInterface::class]
     ) {

Tests/Mock/ExpressionBuilder.php

@@ -112,37 +112,37 @@ public function bitAnd(string $fieldName, int $value): string
         return __FUNCTION__;
     }
 
-    public function min(string $fieldName, string $alias = null): string
+    public function min(string $fieldName, ?string $alias = null): string
     {
         return __FUNCTION__;
     }
 
-    public function max(string $fieldName, string $alias = null): string
+    public function max(string $fieldName, ?string $alias = null): string
     {
         return __FUNCTION__;
     }
 
-    public function avg(string $fieldName, string $alias = null): string
+    public function avg(string $fieldName, ?string $alias = null): string
     {
         return __FUNCTION__;
     }
 
-    public function sum(string $fieldName, string $alias = null): string
+    public function sum(string $fieldName, ?string $alias = null): string
     {
         return __FUNCTION__;
     }
 
-    public function count(string $fieldName, string $alias = null): string
+    public function count(string $fieldName, ?string $alias = null): string
     {
         return __FUNCTION__;
     }
 
-    public function length(string $fieldName, string $alias = null): string
+    public function length(string $fieldName, ?string $alias = null): string
     {
         return __FUNCTION__;
     }
 
-    protected function calculation(string $aggregateName, string $fieldName, string $alias = null): string
+    protected function calculation(string $aggregateName, string $fieldName, ?string $alias = null): string
     {
         return __FUNCTION__;
     }

Tests/Mock/QueryBuilder.php

@@ -20,12 +20,12 @@ public function count(string $item): self
         return $this;
     }
 
-    public function delete(string $delete, string $alias = null): self
+    public function delete(string $delete, ?string $alias = null): self
     {
         return $this;
     }
 
-    public function update(string $update, string $alias = null): self
+    public function update(string $update, ?string $alias = null): self
     {
         return $this;
     }
@@ -61,7 +61,7 @@ public function expr(): ExpressionBuilder
         return $this->expressionBuilder;
     }
 
-    public function createNamedParameter($value, $type = Connection::PARAM_STR, string $placeHolder = null): string
+    public function createNamedParameter($value, $type = Connection::PARAM_STR, ?string $placeHolder = null): string
     {
         return 'p';
     }
@@ -91,7 +91,7 @@ public function addSelectLiteral(string ...$selects): self
         return $this;
     }
 
-    public function from(string $from, string $alias = null): self
+    public function from(string $from, ?string $alias = null): self
     {
         return $this;
     }
@@ -111,7 +111,7 @@ public function setFirstResult(int $firstResult): self
         return $this;
     }
 
-    public function join(string $fromAlias, string $join, string $alias, string $condition = null): self
+    public function join(string $fromAlias, string $join, string $alias, ?string $condition = null): self
     {
         return $this;
     }
@@ -136,12 +136,12 @@ public function groupBy(...$groupBy): self
         return $this;
     }
 
-    public function orderBy(string $fieldName, string $order = null): self
+    public function orderBy(string $fieldName, ?string $order = null): self
     {
         return $this;
     }
 
-    public function addOrderBy(string $fieldName, string $order = null): self
+    public function addOrderBy(string $fieldName, ?string $order = null): self
     {
         return $this;
     }

Tests/Unit/Form/AbstractFormTest.php

@@ -115,7 +115,7 @@ protected function getObjectClassName(): string
     /**
      * @test
      */
-    public function canChainAllChainableSetters(array $chainPropertiesAndValues = null): FormInterface
+    public function canChainAllChainableSetters(?array $chainPropertiesAndValues = null): FormInterface
     {
         if (null === $chainPropertiesAndValues) {
             $chainPropertiesAndValues = $this->chainProperties;