From 39b85c9365350195101de3668e1dea4f23d847fc Mon Sep 17 00:00:00 2001
From: Guy Sartorelli <guy.sartorelli@silverstripe.com>
Date: Tue, 8 Aug 2023 13:04:08 +1200
Subject: [PATCH] MNT Make employee validatable

Fix broken validation that's already there
Add email field for testing field validation
---
 code/Employee.php | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/code/Employee.php b/code/Employee.php
index 0bae537..8caa708 100644
--- a/code/Employee.php
+++ b/code/Employee.php
@@ -3,6 +3,7 @@
 namespace SilverStripe\FrameworkTest\Model;
 
 use SilverStripe\Assets\Image;
+use SilverStripe\Forms\EmailField;
 use SilverStripe\Forms\NumericField;
 use SilverStripe\Forms\TextField;
 use SilverStripe\ORM\Connect\MySQLSchemaManager;
@@ -27,6 +28,7 @@ class Employee extends DataObject
 
     private static $db = array(
         'Name' => 'Varchar',
+        'Email' => 'Varchar',
         'Biography' => 'HTMLText',
         'DateOfBirth' => 'Date',
         'Category' => 'Enum("marketing,management,rnd,hr")'
@@ -90,6 +92,7 @@ class Employee extends DataObject
     {
         // Use basic scaffolder (no tabs)
         $fields = $this->scaffoldFormFields();
+        $fields->replaceField('Email', EmailField::create('Email'));
         $fields->push(new NumericField('ManyMany[YearStart]', 'Year started (3.1, many-many only)'));
         $fields->push(new TextField('ManyMany[Role]', 'Role (3.1, many-many only)'));
         return $fields;
@@ -142,7 +145,7 @@ class Employee extends DataObject
     {
         $result = parent::validate();
         if (!$this->Name) {
-            $result->error('"Name" can\'t be blank');
+            $result->addFieldError('Name', '"Name" can\'t be blank');
         }
         return $result;
     }