Merge pull request #206 from creative-commoners/pulls/2/field-validators

FIX Change type to Int
silverstripe · Nov 6, 2024 · 4ecbf4a · 4ecbf4a
2 parents 0003606 + 020bbb0
commit 4ecbf4a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/code/BasicFieldsTestPage.php b/code/BasicFieldsTestPage.php
@@ -71,7 +71,7 @@ class BasicFieldsTestPage extends TestPage
         'TimeHTML5' => 'Time',
         'ToggleCompositeTextField1' => 'Varchar',
         'ToggleCompositeDropdownField' => 'Varchar',
-        'Validated' => 'Text',
+        'Validated' => 'Int',
     );
 
     private static $has_one = array(
@@ -187,7 +187,7 @@ public function getDefaultData()
             'Time' => "23:59",
             'TimeHTML5' => "23:59",
             'ToggleCompositeTextField1' => 'My value (ä!)',
-            'Validated' => '1',
+            'Validated' => 1,
         );
     }
 

diff --git a/code/Company.php b/code/Company.php
@@ -129,7 +129,7 @@ public function requireDefaultRecords()
             $company = new Company();
             $company->Name = $companyData[0];
             $company->Category = $companyData[1];
-            $company->Revenue = $companyData[2];
+            $company->Revenue = (float) $companyData[2];
             $company->CEO = $companyData[3];
             $company->write();
         }