Merge pull request #158 from creative-commoners/pulls/0.4/toast-msg-g…

…ridfield MNT Add custom validator for test cases
silverstripe · Nov 12, 2023 · 6637e88 · 6637e88
2 parents a5aeaa3 + 3f03425
commit 6637e88
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 14 deletions.
diff --git a/code/Company.php b/code/Company.php
@@ -13,6 +13,7 @@
 use SilverStripe\Versioned\Versioned;
 use RelationFieldsTestPage;
 use GridFieldTestPage;
+use SilverStripe\Forms\RequiredFields;
 
 /**
  *
@@ -100,15 +101,11 @@ public function getCMSFields()
         return $fields;
     }
 
-    public function validate()
+    public function getCMSValidator()
     {
-        $result = parent::validate();
-
-        if (!$this->Name) {
-            $result->addFieldError('Name', 'Name is required');
-        }
-
-        return $result;
+        return new RequiredFields(
+            ['Name']
+        );
     }
 
     public function DynamicProperty()

diff --git a/code/Employee.php b/code/Employee.php
@@ -5,6 +5,7 @@
 use SilverStripe\Assets\Image;
 use SilverStripe\Forms\EmailField;
 use SilverStripe\Forms\NumericField;
+use SilverStripe\Forms\RequiredFields;
 use SilverStripe\Forms\TextField;
 use SilverStripe\ORM\Connect\MySQLSchemaManager;
 use SilverStripe\ORM\DataObject;
@@ -141,13 +142,11 @@ public function requireDefaultRecords()
         srand();
     }
 
-    public function validate()
+    public function getCMSValidator()
     {
-        $result = parent::validate();
-        if (!$this->Name) {
-            $result->addFieldError('Name', '"Name" can\'t be blank');
-        }
-        return $result;
+        return new RequiredFields(
+            ['Name']
+        );
     }
 
     /**