Merge pull request #220 from creative-commoners/pulls/2/validation-in…

…terface API Update validate signature
silverstripe · Dec 17, 2024 · 5989e77 · 5989e77
2 parents 250d5dd + 0bdefa9
commit 5989e77
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/code/BasicFieldsTestPage.php b/code/BasicFieldsTestPage.php
@@ -33,6 +33,7 @@
 use SilverStripe\Forms\CompositeField;
 use SilverStripe\Forms\Validation\RequiredFieldsValidator;
 use SilverStripe\Forms\HTMLReadonlyField;
+use SilverStripe\Core\Validation\ValidationResult;
 
 class BasicFieldsTestPage extends TestPage
 {
@@ -402,7 +403,7 @@ public function getCMSValidator()
         return new RequiredFieldsValidator('Required');
     }
 
-    public function validate()
+    public function validate(): ValidationResult
     {
         $result = parent::validate();
         if (!$this->Validated || $this->Validated < 1 || $this->Validated > 3) {

diff --git a/code/multitab-validation/MultiTabPage.php b/code/multitab-validation/MultiTabPage.php
@@ -6,6 +6,7 @@
 use SilverStripe\Forms\EmailField;
 use SilverStripe\Forms\Validation\RequiredFieldsValidator;
 use SilverStripe\Forms\TextField;
+use SilverStripe\Core\Validation\ValidationResult;
 
 if (!class_exists(Page::class)) {
     return;
@@ -59,7 +60,7 @@ public function getCMSValidator()
         ]);
     }
 
-    public function validate()
+    public function validate(): ValidationResult
     {
         $result = parent::validate();