NEW DBField validation

_config/model.yml

@@ -20,6 +20,8 @@ SilverStripe\Core\Injector\Injector:
     class: SilverStripe\ORM\FieldType\DBDecimal
   Double:
     class: SilverStripe\ORM\FieldType\DBDouble
+  Email:
+    class: SilverStripe\ORM\FieldType\DBEmail
   Enum:
     class: SilverStripe\ORM\FieldType\DBEnum
   Float:

src/Control/Email/Email.php

@@ -13,6 +13,7 @@
 use SilverStripe\Core\Injector\Injectable;
 use SilverStripe\Core\Injector\Injector;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 use SilverStripe\Model\ArrayData;
 use SilverStripe\View\Requirements;
 use SilverStripe\View\SSViewer;
@@ -509,7 +510,7 @@ private function updateHtmlAndTextWithRenderedTemplates(bool $plainOnly = false)
         // Plain render fallbacks to using the html render with html tags removed
         if (!$plainRender && $htmlRender) {
             // call html_entity_decode() to ensure any encoded HTML is also stripped inside ->Plain()
-            $dbField = DBField::create_field('HTMLFragment', html_entity_decode($htmlRender));
+            $dbField = DBFieldHelper::create_field('HTMLFragment', html_entity_decode($htmlRender));
             $plainRender = $dbField->Plain();
         }
 

src/Control/RSS/RSSFeed_Entry.php

@@ -4,6 +4,7 @@
 
 use SilverStripe\Control\Director;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 use SilverStripe\ORM\FieldType\DBHTMLText;
 use SilverStripe\Model\ModelData;
 use BadMethodCallException;

src/Forms/ConfirmedPasswordField.php

@@ -6,6 +6,7 @@
 use SilverStripe\ORM\DataObject;
 use SilverStripe\ORM\DataObjectInterface;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 use SilverStripe\Security\Authenticator;
 use SilverStripe\Security\Security;
 use SilverStripe\View\HTML;
@@ -771,6 +772,6 @@ private function getAttributesHTMLForChild(FormField $child): DBField
         if (strpos($attributes, 'required="required"') === false && $this->Required()) {
             $attributes .= ' required="required" aria-required="true"';
         }
-        return DBField::create_field('HTMLFragment', $attributes);
+        return DBFieldHelper::create_field('HTMLFragment', $attributes);
     }
 }

src/Forms/EmailField.php

@@ -2,6 +2,9 @@
 
 namespace SilverStripe\Forms;
 
+use SilverStripe\Core\Validation\ConstraintValidator;
+use Symfony\Component\Validator\Constraints;
+
 /**
  * Text input field with validation for correct email format according to RFC 2822.
  */
@@ -18,32 +21,27 @@ public function Type()
     }
 
     /**
-     * Validates for RFC 2822 compliant email addresses.
-     *
-     * @see http://www.regular-expressions.info/email.html
-     * @see http://www.ietf.org/rfc/rfc2822.txt
-     *
      * @param Validator $validator
      *
      * @return string
      */
     public function validate($validator)
     {
-        $result = true;
         $this->value = trim($this->value ?? '');
 
-        $pattern = '^[a-z0-9!#$%&\'*+/=?^_`{|}~-]+(?:\\.[a-z0-9!#$%&\'*+/=?^_`{|}~-]+)*@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?$';
-
-        // Escape delimiter characters.
-        $safePattern = str_replace('/', '\\/', $pattern ?? '');
-
-        if ($this->value && !preg_match('/' . $safePattern . '/i', $this->value ?? '')) {
+        $result = true;
+        // $message = _t('SilverStripe\\Forms\\EmailField.VALIDATION', 'Please enter an email address');
+        // $validationResult = ConstraintValidator::validate(
+        //     $this->value,
+        //     new Constraints\Email(message: $message)
+        // );
+        $validationResult = $this->getModelField()->validate();
+
+        if (!$validationResult->isValid()) {
             $validator->validationError(
                 $this->name,
-                _t('SilverStripe\\Forms\\EmailField.VALIDATION', 'Please enter an email address'),
-                'validation'
+                $validationResult->getMessages()[0]['message'],
             );
-
             $result = false;
         }
 

src/Forms/FormField.php

@@ -10,11 +10,16 @@
 use SilverStripe\Core\Convert;
 use SilverStripe\ORM\DataObjectInterface;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 use SilverStripe\ORM\FieldType\DBHTMLText;
 use SilverStripe\Core\Validation\ValidationResult;
 use SilverStripe\View\AttributesHTML;
 use SilverStripe\View\SSViewer;
 use SilverStripe\Model\ModelData;
+use SilverStripe\Model\ModelFields\ModelField;
+use SilverStripe\Model\ModelFields\ModelFieldTrait;
+use SilverStripe\Model\ModelFields\StringModelField;
+use SilverStripe\Core\Injector\Injector;
 
 /**
  * Represents a field in a form.
@@ -40,11 +45,28 @@
  * including both structure (name, id, attributes, etc.) and state (field value).
  * Can be used by for JSON data which is consumed by a front-end application.
  */
-class FormField extends RequestHandler
+abstract class FormField extends RequestHandler
 {
     use AttributesHTML;
     use FormMessage;
 
+    // todo: possibly should make this abstract
+    protected string $modelFieldClass = StringModelField::class;
+
+    private ModelField $modelField;
+
+    protected function getModelField(): ModelField
+    {
+        return $this->modelField;
+    }
+
+    private function initModelField(): void
+    {
+        $name = $this->getName();
+        $modelField = Injector::inst()->createWithArgs($this->modelFieldClass, [$name]);
+        $this->modelField = $modelField;
+    }
+
     /** @see $schemaDataType */
     const SCHEMA_DATA_TYPE_STRING = 'String';
 
@@ -344,6 +366,7 @@ public function __construct($name, $title = null, $value = null)
 
         parent::__construct();
 
+        $this->initModelField($name);
         $this->setupDefaultClasses();
     }
 
@@ -929,7 +952,7 @@ public function Field($properties = [])
         // Trim whitespace from the result, so that trailing newlines are suppressed. Works for strings and HTMLText values
         if (is_string($result)) {
             $result = trim($result ?? '');
-        } elseif ($result instanceof DBField) {
+        } elseif ($result instanceof ModelField) {
             $result->setValue(trim($result->getValue() ?? ''));
         }
 
@@ -1231,9 +1254,12 @@ protected function extendValidationResult(bool $result, Validator $validator): b
     }
 
     /**
-     * Abstract method each {@link FormField} subclass must implement, determines whether the field
+     * Method each {@link FormField} subclass can implement, determines whether the field
      * is valid or not based on the value.
      *
+     * Subclass methods should call $this->extendValidationResult(true, $validator)
+     * at the end of the method
+     *
      * @param Validator $validator
      * @return bool
      */

src/Forms/FormRequestHandler.php

@@ -216,10 +216,10 @@ public function httpSubmission($request)
         // Action handlers may throw ValidationExceptions.
         try {
             // Or we can use the Validator attached to the form
-            $result = $this->form->validationResult();
-            if (!$result->isValid()) {
-                return $this->getValidationErrorResponse($result);
-            }
+            // $result = $this->form->validationResult();
+            // if (!$result->isValid()) {
+            //     return $this->getValidationErrorResponse($result);
+            // }
 
             // First, try a handler method on the controller (has been checked for allowed_actions above already)
             $controller = $this->form->getController();

src/Forms/GridField/GridField.php

@@ -22,6 +22,7 @@
 use SilverStripe\ORM\DataList;
 use SilverStripe\ORM\DataObjectInterface;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 use SilverStripe\Model\List\Filterable;
 use SilverStripe\Model\List\Limitable;
 use SilverStripe\Model\List\Sortable;
@@ -382,14 +383,14 @@ public function getCastedValue($value, $castingDefinition)
 
         if (strpos($castingDefinition ?? '', '->') === false) {
             $castingFieldType = $castingDefinition;
-            $castingField = DBField::create_field($castingFieldType, $value);
+            $castingField = DBFieldHelper::create_field($castingFieldType, $value);
 
             return call_user_func_array([$castingField, 'XML'], $castingParams ?? []);
         }
 
         list($castingFieldType, $castingMethod) = explode('->', $castingDefinition ?? '');
 
-        $castingField = DBField::create_field($castingFieldType, $value);
+        $castingField = DBFieldHelper::create_field($castingFieldType, $value);
 
         return call_user_func_array([$castingField, $castingMethod], $castingParams ?? []);
     }
@@ -543,6 +544,8 @@ public function FieldHolder($properties = [])
             'footer' => '',
         ];
 
+        $_c = $this->getComponents();
+
         foreach ($this->getComponents() as $item) {
             if ($item instanceof GridField_HTMLProvider) {
                 $fragments = $item->getHTMLFragments($this);

src/Forms/GridField/GridFieldLevelup.php

@@ -5,6 +5,7 @@
 use LogicException;
 use SilverStripe\ORM\DataObject;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 use SilverStripe\ORM\Hierarchy\Hierarchy;
 use SilverStripe\Model\ArrayData;
 use SilverStripe\View\HTML;
@@ -91,7 +92,7 @@ public function getHTMLFragments($gridField)
         $linkTag = HTML::createTag('a', $attrs);
 
         $forTemplate = new ArrayData([
-            'UpLink' => DBField::create_field('HTMLFragment', $linkTag)
+            'UpLink' => DBFieldHelper::create_field('HTMLFragment', $linkTag)
         ]);
 
         $template = SSViewer::get_templates_by_class($this, '', __CLASS__);

src/Forms/LookupField.php

@@ -6,6 +6,7 @@
 use SilverStripe\Core\ArrayLib;
 use SilverStripe\ORM\DataObjectInterface;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 
 /**
  * Read-only complement of {@link MultiSelectField}.
@@ -58,7 +59,7 @@ public function Field($properties = [])
         }
 
         $properties = array_merge($properties, [
-            'AttrValue' => DBField::create_field('HTMLFragment', $attrValue),
+            'AttrValue' => DBFieldHelper::create_field('HTMLFragment', $attrValue),
             'InputValue' => $inputValue
         ]);
 

src/Forms/SearchableLookupField.php

@@ -6,6 +6,7 @@
 use SilverStripe\Core\ArrayLib;
 use SilverStripe\ORM\DataObjectInterface;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 use SilverStripe\Model\List\ArrayList;
 use SilverStripe\ORM\DataList;
 

src/Forms/SelectionGroup.php

@@ -4,6 +4,7 @@
 
 use SilverStripe\Model\List\ArrayList;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 use SilverStripe\View\HTML;
 
 /**
@@ -91,7 +92,7 @@ public function FieldList()
 
             $itemID = $this->ID() . '_' . (++$count);
             $extra = [
-                "RadioButton" => DBField::create_field('HTMLFragment', HTML::createTag(
+                "RadioButton" => DBFieldHelper::create_field('HTMLFragment', HTML::createTag(
                     'input',
                     [
                         'class' => 'selector',

src/Forms/SingleLookupField.php

@@ -6,6 +6,7 @@
 use SilverStripe\ORM\DataObjectInterface;
 use SilverStripe\Model\List\Map;
 use SilverStripe\ORM\FieldType\DBField;
+use SilverStripe\ORM\FieldType\DBFieldHelper;
 
 /**
  * Read-only complement of {@link DropdownField}.
@@ -125,7 +126,7 @@ public function Field($properties = [])
         }
 
         $properties = array_merge($properties, [
-            'AttrValue' => DBField::create_field('HTMLFragment', $attrValue),
+            'AttrValue' => DBFieldHelper::create_field('HTMLFragment', $attrValue),
             'InputValue' => $inputValue
         ]);
 

src/Forms/TextField.php

@@ -2,11 +2,15 @@
 
 namespace SilverStripe\Forms;
 
+use SilverStripe\Model\ModelFields\StringModelField;
+
 /**
  * Text input field.
  */
 class TextField extends FormField implements TippableFieldInterface
 {
+    protected string $modelFieldClass = StringModelField::class;
+
     /**
      * @var int
      */