API Use updated validation API

silverstripe · Nov 11, 2024 · 166ec67 · 166ec67
1 parent 16a447e
commit 166ec67
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/src/EditableSpamProtectionField.php b/src/EditableSpamProtectionField.php
@@ -227,9 +227,9 @@ public function validateField($data, $form)
             $formField->setValue($data[$this->Name]);
         }
 
-        $validator = $form->getValidator();
-        if (!$formField->validate($validator)) {
-            $errors = $validator->getErrors();
+        $result = $formField->validate();
+        if (!$result->isValid()) {
+            $errors = $result->getMessages();
             $foundError = false;
 
             // field validate implementation may not add error to validator

diff --git a/tests/EditableSpamProtectionFieldTest.php b/tests/EditableSpamProtectionFieldTest.php
@@ -14,6 +14,7 @@
 use SilverStripe\SpamProtection\EditableSpamProtectionField;
 use SilverStripe\SpamProtection\Extension\FormSpamProtectionExtension;
 use SilverStripe\SpamProtection\Tests\Stub\Protector;
+use SilverStripe\Core\Validation\ValidationResult;
 
 class EditableSpamProtectionFieldTest extends SapphireTest
 {
@@ -39,11 +40,12 @@ public function testValidateFieldDoesntAddErrorOnSuccess()
         $formMock = $this->getFormMock();
         $formFieldMock = $this->getEditableFormFieldMock();
 
+        $result = new ValidationResult;
         $formFieldMock
             ->getFormField() // mock
             ->expects($this->once())
             ->method('validate')
-            ->willReturn(true);
+            ->willReturn($result);
 
         $formMock
             ->expects($this->never())
@@ -57,11 +59,13 @@ public function testValidateFieldAddsErrorFromField()
         $formMock = $this->getFormMock();
         $formFieldMock = $this->getEditableFormFieldMock();
 
+        $result = new ValidationResult;
+        $result->addError('fail');
         $formFieldMock
             ->getFormField() // mock
             ->expects($this->once())
             ->method('validate')
-            ->willReturn(false);
+            ->willReturn($result);
 
         $formMock->getValidator()->validationError('MyField', 'some field message', 'required');
 
@@ -78,11 +82,13 @@ public function testValidateFieldAddsDefaultError()
         $formMock = $this->getFormMock();
         $formFieldMock = $this->getEditableFormFieldMock();
 
+        $result = new ValidationResult;
+        $result->addError('fail');
         $formFieldMock
             ->getFormField() // mock
             ->expects($this->once())
             ->method('validate')
-            ->willReturn(false);
+            ->willReturn($result);
 
         // field doesn't set any validation errors here