tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-06-28 15:39:40 +02:00
Code Issues Projects Releases Wiki Activity

BUG Prevent dev/build continually regenerating Number field type

Browse Source
This commit is contained in:
Damian Mooyman 2015-11-02 13:59:45 +13:00
parent 583cbdd77f
commit 0272e443f4
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
model/connect/MySQLSchemaManager.php
View File

@ -407,15 +407,17 @@ class MySQLSchemaManager extends DBSchemaManager {
$precision = $values['precision'];
}
$defaultValue = '';
// Fix format of default value to match precision
if (isset($values['default']) && is_numeric($values['default'])) {
$decs = strpos($precision, ',') !== false
? (int) substr($precision, strpos($precision, ',') + 1)
: 0;
$defaultValue = ' default ' . number_format($values['default'], $decs, '.', '');
$values['default'] = number_format($values['default'], $decs, '.', '');
} else {
unset($values['default']);
}
return "decimal($precision) not null $defaultValue";
return "decimal($precision) not null" . $this->defaultClause($values);
}
/**

5
tests/model/DataObjectSchemaGenerationTest.php
View File

@ -178,6 +178,11 @@ class DataObjectSchemaGenerationTest_DO extends DataObject implements TestOnly {
private static $db = array(
'Enum1' => 'Enum("A, B, C, D","")',
'Enum2' => 'Enum("A, B, C, D","A")',
'NumberField' => 'Decimal',
'FloatingField' => 'Decimal(10,3,1.1)',
'TextValue' => 'Varchar',
'Date' => 'SS_Datetime',
'MyNumber' => 'Int'
);
}