tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-06 16:18:41 +02:00
Code Issues Projects Releases Wiki Activity

Update datamodel.md to use an array argument to Enum instead of the more error-prone string

Browse Source 
The previous argument to the Enum was: 
"Status" => "Enum('Active, Injured, Retired')"

This would cause issues where the 'Injured' and 'Retired' statuses would actually be stored in the database as ' Injured' and ' Retired' respectively, at least in 3.0.

Using an array() to specify the enum, and using normal array syntax (albeit within a string) is less likely to cause errors.
This commit is contained in:
madmatt 2013-10-05 00:53:08 +13:00
parent 902c8db859
commit 951152e1c6
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
docs/en/topics/datamodel.md
View File

@ -375,7 +375,7 @@ Data is defined in the static variable $db on each class, in the format:
"FirstName" => "Varchar",
"Surname" => "Varchar",
"Description" => "Text",
"Status" => "Enum('Active, Injured, Retired')",
"Status" => "Enum(array('Active', 'Injured', 'Retired'))",
"Birthday" => "Date"
);
}
@ -393,7 +393,7 @@ the default behavior by making a function called "get`<fieldname>`" or
:::php
class Player extends DataObject {
private static $db = array(
"Status" => "Enum('Active, Injured, Retired')"
"Status" => "Enum(array('Active', 'Injured', 'Retired'))"
);
// access through $myPlayer->Status