API Make parameter non-optional for PHP 8.4

silverstripe · Dec 1, 2024 · e5735f9 · e5735f9
1 parent dbe7b72
commit e5735f9
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/code/Forms/FileHistoryFormFactory.php b/code/Forms/FileHistoryFormFactory.php
@@ -47,7 +47,7 @@ protected function getSpecsMarkup($record)
         );
     }
 
-    protected function getFormFields(?RequestHandler $controller = null, $name, $context = [])
+    protected function getFormFields(?RequestHandler $controller, $name, $context = [])
     {
         $record = $context['Record'];
 

diff --git a/code/Forms/FileSearchFormFactory.php b/code/Forms/FileSearchFormFactory.php
@@ -49,7 +49,7 @@ public function getForm(?RequestHandler $controller = null, $name = FormFactory:
      * @param array $context
      * @return FieldList
      */
-    protected function getFormFields(?RequestHandler $controller = null, $name, $context = [])
+    protected function getFormFields(?RequestHandler $controller, $name, $context = [])
     {
         // Note: "Name" field is excluded as it is baked directly into the Search.js react component
 

diff --git a/code/Forms/FolderCreateFormFactory.php b/code/Forms/FolderCreateFormFactory.php
@@ -16,7 +16,7 @@ public function getRequiredContext()
         return ['ParentID'];
     }
 
-    public function getFormFields(?RequestHandler $controller = null, $name, $context = [])
+    public function getFormFields(?RequestHandler $controller, $name, $context = [])
     {
         // Add status flag before extensions are triggered
         $this->beforeExtending('updateFormFields', function (FieldList $fields) use ($context) {

diff --git a/code/Forms/FolderFormFactory.php b/code/Forms/FolderFormFactory.php
@@ -13,7 +13,7 @@
 class FolderFormFactory extends AssetFormFactory
 {
 
-    protected function getFormFields(?RequestHandler $controller = null, $name, $context = [])
+    protected function getFormFields(?RequestHandler $controller, $name, $context = [])
     {
         /** @var Folder $record */
         $record = $context['Record'] ?? null;