Browse Source

Use `hasAttribute()` instead to avoid code duplication

tags/2.0.0-alpha
Niko Wicaksono 11 years ago
parent
commit
a042d20c22
  1. 4
      framework/yii/db/ActiveRecord.php

4
framework/yii/db/ActiveRecord.php

@ -399,7 +399,7 @@ class ActiveRecord extends Model
*/ */
public function __set($name, $value) public function __set($name, $value)
{ {
if (isset($this->_attributes[$name]) || isset($this->getTableSchema()->columns[$name])) { if ($this->hasAttribute($name)) {
$this->_attributes[$name] = $value; $this->_attributes[$name] = $value;
} else { } else {
parent::__set($name, $value); parent::__set($name, $value);
@ -562,7 +562,7 @@ class ActiveRecord extends Model
*/ */
public function setAttribute($name, $value) public function setAttribute($name, $value)
{ {
if (isset($this->_attributes[$name]) || isset($this->getTableSchema()->columns[$name])) { if ($this->hasAttribute($name)) {
$this->_attributes[$name] = $value; $this->_attributes[$name] = $value;
} else { } else {
throw new InvalidParamException(get_class($this) . ' has no attribute named "' . $name . '".'); throw new InvalidParamException(get_class($this) . ' has no attribute named "' . $name . '".');

Loading…
Cancel
Save