Save() Does Not Work Due To (Unrelated?) Rule For Other Column

Yii 1.1 General Discussion
1

Hello everybody,

having a little issue here. I wanna update the field ‘lastactive’ field in my database after a successful login. This is my changed UserIdentity::authenticate() method:




private $_id;

 

public function authenticate()

{

	// $this->username === provided email in call

	$dbuser=User::model()->findByAttributes(array('email'=>$this->username));

	if($dbuser === null) // no user was found in the database

		$this->errorCode=self::ERROR_USERNAME_INVALID;

	if(!CPasswordHelper::verifyPassword($this->password, $dbuser->password)) // wrong password

		$this->errorCode=self::ERROR_PASSWORD_INVALID;

	else { // OK!

		$this->setLoginAttributes($dbuser);

		$this->errorCode=self::ERROR_NONE;

	}

	return !$this->errorCode;

}


public function setLoginAttributes($dbuser)

{

	$this->_id = $dbuser->id;

	$this->setState('first_name', $dbuser->first_name);

	$this->setState('last_name', $dbuser->last_name);

	$this->setState('status', $dbuser->status);

	$dbuser->lastactive = New CDbExpression('NOW()');

	$dbuser->save(array('lastactive'));

}


public function getId()

{

	return $this->_id;

}

But the save operation doesn’t work. So I checked if it does with SaveAttributes(array(‘lastactive’)), and it does. So I assumed it’s a problem with my validation rules. I checked them one after another, and turned out the one who causes the problem is the fourth from the top about my birthday column:




public function rules()

{

	// NOTE: you should only define rules for those attributes that

	// will receive user inputs.

	return array(

		array('email, password', 'required'),

		array('email', 'email'),

		array('email', 'unique'),

		array('birthday', 'date'),

		array('active, online, address_id', 'numerical', 'integerOnly'=>true),

		array('first_name, last_name, email, image', 'length', 'max'=>255),

		array('about', 'length', 'max'=>4000),

		array('status', 'length', 'max'=><img src='http://www.yiiframework.com/forum/public/style_emoticons/default/cool.gif' class='bbc_emoticon' alt='8)' />,

		//array('password', 'length', 'is'=>60),

		array('birthday, joined, lastactive', 'safe'),

		// The following rule is used by search().

		// @todo Please remove those attributes that should not be searched.

		array('id, first_name, last_name, email, birthday, about, joined, lastactive, status, active, image, online, password, address_id', 'safe', 'on'=>'search'),

	);

}

Can somebody explain me why the reason for that?

Thanks in advance!

2

So, I’ve found out what my problem is: The column defaults to yyyy-MM-dd (MySQL default), while the CDateValidator defaults to dd-MM-yyyy (I think). So I changed my rule to




array('birthday', 'date', 'format' => 'yyyy-MM-dd'),

and now it works fine. I think the problem was that during registration I didn’t give my user a birthday, and thus MySQL defaulted to 0000-00-00. So, on update, Yii noticed that my entry was in the wrong format. Still I’m kinda confused: Does Yii check every attribute before saving, although I told it to only save the ‘lastactive’ entry?

Cheers!

3

I’m going to go with ‘’ is no a valid date :)

I think what is going on is that $model->save() goes through validation. $model->updateByAtributes() doesn’t.

For what you are doing the byAttribute route is better. The SQL generated is cleaner. Not much of a problem on a small site, but if it grows…

4

Thanks a lot for your answer! I already found out that it’s rules related, see my previous post. Still confused about it, but I only started out with Yii a couple of weeks ago ;) Could you maybe elaborate on the UpdateByAttributes() suggestion? Can’t find it in the CActiveRecord class reference, but there seems to be update(), updateAll() and updateByPk().

Thanks again!

5

I was explaining ‘why’ you might be getting the error. updateByAttribute was OTMH, What I meant was saveAtribute(),




        $dbuser->lastactive = New CDbExpression('NOW()');

        $dbuser->saveAttributes(array('lastactive'));

From the Docs

6

Thanks, I went with update(‘lastactive’) now and it works just fine (used SaveAttributes before and did the job as well, see my OP).