Commit 27943ce7 by Carsten Brandt

Merge pull request #4063 from yiisoft/typecast-split

split typecast method into two
parents aff3a41e 5d0452b2
......@@ -621,9 +621,9 @@ abstract class ActiveRecord extends BaseActiveRecord
if (isset($columns[$name])) {
if ($columns[$name]->isMva) {
$mvaValue = explode(',', $value);
$row[$name] = array_map(array($columns[$name], 'typecast'), $mvaValue);
$row[$name] = array_map(array($columns[$name], 'phpTypecast'), $mvaValue);
} else {
$row[$name] = $columns[$name]->typecast($value);
$row[$name] = $columns[$name]->phpTypecast($value);
}
}
}
......
......@@ -8,6 +8,7 @@ Yii Framework 2 sphinx extension Change Log
- Bug #4018: AR relation eager loading does not work with db models (klimov-paul)
- Enh: Added support for using sub-queries when building a DB query with `IN` condition (qiangxue)
- Enh #3520: Added `unlinkAll()`-method to active record to remove all records of a model relation (NmDimas, samdark, cebe)
- Chg #2287: Split `yii\sphinx\ColumnSchema::typecast()` into two methods `phpTypecast()` and `dbTypecast()` to allow specifying PDO type explicitly (cebe)
2.0.0-beta April 13, 2014
......
......@@ -55,12 +55,12 @@ class ColumnSchema extends Object
public $isMva;
/**
* Converts the input value according to [[phpType]].
* Converts the input value according to [[phpType]] after retrieval from the database.
* If the value is null or an [[Expression]], it will not be converted.
* @param mixed $value input value
* @return mixed converted value
*/
public function typecast($value)
public function phpTypecast($value)
{
if ($value === null || gettype($value) === $this->phpType || $value instanceof Expression) {
return $value;
......@@ -70,13 +70,29 @@ class ColumnSchema extends Object
}
switch ($this->phpType) {
case 'string':
return (string) $value;
return is_resource($value) ? $value : (string) $value;
case 'integer':
return (integer) $value;
case 'boolean':
return (boolean) $value;
case 'double':
return (double) $value;
}
return $value;
}
/**
* Converts the input value according to [[type]] and [[dbType]] for use in a db query.
* If the value is null or an [[Expression]], it will not be converted.
* @param mixed $value input value
* @return mixed converted value. This may also be an array containing the value as the first element
* and the PDO type as the second element.
*/
public function dbTypecast($value)
{
// the default implementation does the same as casting for PHP but it should be possible
// to override this with annotation of explicit PDO type.
return $this->phpTypecast($value);
}
}
......@@ -973,14 +973,14 @@ class QueryBuilder extends Object
} else {
$phName = self::PARAM_PREFIX . count($params);
$lineParts[] = $phName;
$params[$phName] = (isset($columnSchema)) ? $columnSchema->typecast($subValue) : $subValue;
$params[$phName] = (isset($columnSchema)) ? $columnSchema->dbTypecast($subValue) : $subValue;
}
}
return '(' . implode(',', $lineParts) . ')';
} else {
$phName = self::PARAM_PREFIX . count($params);
$params[$phName] = (isset($columnSchema)) ? $columnSchema->typecast($value) : $value;
$params[$phName] = (isset($columnSchema)) ? $columnSchema->dbTypecast($value) : $value;
return $phName;
}
......
......@@ -131,6 +131,7 @@ Yii Framework 2 Change Log
- Enh: Improved `yii\helpers\Inflector::slug` to support more cases for Russian, Hebrew and special characters (samdark)
- Enh #3926: `yii\widgets\Breadcrumbs::$links`. Allows individual link to have its own `template` (creocoder, umneeq)
- Enh #4028: Added ability to `yii\widgets\Menu` to encode each item's label separately (creocoder, umneeq)
- Chg #2287: Split `yii\db\ColumnSchema::typecast()` into two methods `phpTypecast()` and `dbTypecast()` to allow specifying PDO type explicitly (cebe)
- Chg #2898: `yii\console\controllers\AssetController` is now using hashes instead of timestamps (samdark)
- Chg #2913: RBAC `DbManager` is now initialized via migration (samdark)
- Chg #3036: Upgraded Twitter Bootstrap to 3.1.x (qiangxue)
......
......@@ -357,7 +357,7 @@ class ActiveRecord extends BaseActiveRecord
$columns = static::getTableSchema()->columns;
foreach ($row as $name => $value) {
if (isset($columns[$name])) {
$row[$name] = $columns[$name]->typecast($value);
$row[$name] = $columns[$name]->phpTypecast($value);
}
}
parent::populateRecord($record, $row);
......@@ -454,7 +454,7 @@ class ActiveRecord extends BaseActiveRecord
if ($table->sequenceName !== null) {
foreach ($table->primaryKey as $name) {
if ($this->getAttribute($name) === null) {
$id = $table->columns[$name]->typecast($db->getLastInsertID($table->sequenceName));
$id = $table->columns[$name]->phpTypecast($db->getLastInsertID($table->sequenceName));
$this->setAttribute($name, $id);
$values[$name] = $id;
break;
......
......@@ -33,7 +33,7 @@ class ColumnSchema extends Object
public $type;
/**
* @var string the PHP type of this column. Possible PHP types include:
* string, boolean, integer, double.
* `string`, `boolean`, `integer`, `double`.
*/
public $phpType;
/**
......@@ -79,12 +79,12 @@ class ColumnSchema extends Object
public $comment;
/**
* Converts the input value according to [[phpType]].
* Converts the input value according to [[phpType]] after retrieval from the database.
* If the value is null or an [[Expression]], it will not be converted.
* @param mixed $value input value
* @return mixed converted value
*/
public function typecast($value)
public function phpTypecast($value)
{
if ($value === '' && $this->type !== Schema::TYPE_TEXT && $this->type !== Schema::TYPE_STRING && $this->type !== Schema::TYPE_BINARY) {
return null;
......@@ -106,4 +106,18 @@ class ColumnSchema extends Object
return $value;
}
/**
* Converts the input value according to [[type]] and [[dbType]] for use in a db query.
* If the value is null or an [[Expression]], it will not be converted.
* @param mixed $value input value
* @return mixed converted value. This may also be an array containing the value as the first element
* and the PDO type as the second element.
*/
public function dbTypecast($value)
{
// the default implementation does the same as casting for PHP but it should be possible
// to override this with annotation of explicit PDO type.
return $this->phpTypecast($value);
}
}
......@@ -147,7 +147,7 @@ class QueryBuilder extends \yii\base\Object
} else {
$phName = self::PARAM_PREFIX . count($params);
$placeholders[] = $phName;
$params[$phName] = !is_array($value) && isset($columnSchemas[$name]) ? $columnSchemas[$name]->typecast($value) : $value;
$params[$phName] = !is_array($value) && isset($columnSchemas[$name]) ? $columnSchemas[$name]->dbTypecast($value) : $value;
}
}
......@@ -188,7 +188,7 @@ class QueryBuilder extends \yii\base\Object
$vs = [];
foreach ($row as $i => $value) {
if (!is_array($value) && isset($columnSchemas[$columns[$i]])) {
$value = $columnSchemas[$columns[$i]]->typecast($value);
$value = $columnSchemas[$columns[$i]]->dbTypecast($value);
}
if (is_string($value)) {
$value = $this->db->quoteValue($value);
......@@ -247,7 +247,7 @@ class QueryBuilder extends \yii\base\Object
} else {
$phName = self::PARAM_PREFIX . count($params);
$lines[] = $this->db->quoteColumnName($name) . '=' . $phName;
$params[$phName] = !is_array($value) && isset($columnSchemas[$name]) ? $columnSchemas[$name]->typecast($value) : $value;
$params[$phName] = !is_array($value) && isset($columnSchemas[$name]) ? $columnSchemas[$name]->dbTypecast($value) : $value;
}
}
......
......@@ -251,7 +251,7 @@ class Schema extends \yii\db\Schema
} elseif (isset($type) && $type === 'bit') {
$column->defaultValue = hexdec(trim($info['Default'],'X\''));
} else {
$column->defaultValue = $column->typecast($info['Default']);
$column->defaultValue = $column->phpTypecast($info['Default']);
}
return $column;
......
......@@ -222,7 +222,7 @@ class Schema extends \yii\db\Schema
$info['column_default'] = null;
}
if (!$column->isPrimaryKey && ($column->type !== 'timestamp' || $info['column_default'] !== 'CURRENT_TIMESTAMP')) {
$column->defaultValue = $column->typecast($info['column_default']);
$column->defaultValue = $column->phpTypecast($info['column_default']);
}
return $column;
......
......@@ -177,7 +177,7 @@ class Schema extends \yii\db\Schema
} elseif (isset($type) && $type === 'bit') {
$column->defaultValue = bindec(trim($info['Default'],'b\''));
} else {
$column->defaultValue = $column->typecast($info['Default']);
$column->defaultValue = $column->phpTypecast($info['Default']);
}
}
......
......@@ -223,7 +223,7 @@ EOD;
if (stripos($column['DATA_DEFAULT'], 'timestamp') !== false) {
$c->defaultValue = null;
} else {
$c->defaultValue = $c->typecast($column['DATA_DEFAULT']);
$c->defaultValue = $c->phpTypecast($column['DATA_DEFAULT']);
}
}
......
......@@ -416,9 +416,9 @@ SQL;
} elseif (preg_match("/^'(.*?)'::/", $column->defaultValue, $matches)) {
$column->defaultValue = $matches[1];
} elseif (preg_match("/^(.*?)::/", $column->defaultValue, $matches)) {
$column->defaultValue = $column->typecast($matches[1]);
$column->defaultValue = $column->phpTypecast($matches[1]);
} else {
$column->defaultValue = $column->typecast($column->defaultValue);
$column->defaultValue = $column->phpTypecast($column->defaultValue);
}
}
}
......
......@@ -80,7 +80,7 @@ class QueryBuilder extends \yii\db\QueryBuilder
$vs = [];
foreach ($row as $i => $value) {
if (!is_array($value) && isset($columnSchemas[$columns[$i]])) {
$value = $columnSchemas[$columns[$i]]->typecast($value);
$value = $columnSchemas[$columns[$i]]->dbTypecast($value);
}
if (is_string($value)) {
$value = $this->db->quoteValue($value);
......
......@@ -252,7 +252,7 @@ class Schema extends \yii\db\Schema
$column->defaultValue = new Expression('CURRENT_TIMESTAMP');
} else {
$value = trim($info['dflt_value'], "'\"");
$column->defaultValue = $column->typecast($value);
$column->defaultValue = $column->phpTypecast($value);
}
}
......
......@@ -50,6 +50,6 @@ class ColumnSchemaTest extends SphinxTestCase
$columnSchema = new ColumnSchema();
$columnSchema->type = $type;
$columnSchema->phpType = $phpType;
$this->assertEquals($expectedResult, $columnSchema->typecast($value));
$this->assertEquals($expectedResult, $columnSchema->phpTypecast($value));
}
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment