[BUGFIX] Passing null to parameter of type string is deprecated, fixes …

…#247
tpwd · Oct 25, 2024 · 37d0f3e · 37d0f3e
1 parent f0f858b
commit 37d0f3e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/ChangeLog b/ChangeLog
@@ -1,5 +1,8 @@
 ChangeLog
 
+Upcoming version
+[BUGFIX]  parse_ini_string(): Passing null to parameter #1 ($ini_string) of type string is deprecated. Thanks to medarob. https://github.com/tpwd/ke_search/issues/247
+
 Version 6.0.1, 25 October 2024
 [BUGFIX]  Add missing storage page and recursive option. Thanks to Torben Hansen. https://github.com/tpwd/ke_search/issues/251
 [BUGFIX]  Undefined array key "sortByAdmin". Thanks to Simon Praetorius. https://github.com/tpwd/ke_search/issues/252

diff --git a/Classes/Service/AdditionalContentService.php b/Classes/Service/AdditionalContentService.php
@@ -82,7 +82,7 @@ protected function parseAndProcessAdditionalTablesConfiguration(): array
         // parse_ini_string will throw a warning if it could not parse the string.
         // If the system is configured to turn a warning into an exception we catch it here.
         try {
-            $additionalTableConfig = parse_ini_string($this->indexerConfig['additional_tables'], true);
+            $additionalTableConfig = parse_ini_string($this->indexerConfig['additional_tables'] ?? '', true);
         } catch (\Exception $e) {
             $errorMessage =
                 'Error while parsing additional table configuration for indexer "' . $this->indexerConfig['title']