help with update action

Yii 1.1 General Discussion
1

EDIT: I was running into problems until post#9&10 that was preventing getting anyway with what I want. With that now fixed I still want to be able to set an attribute’s (activated) date on an update action only when the status changes from ‘I’ to ‘A’. So I need to be able to get the current value in the db and compare it with the form value and combine that into my if statement within the controller. Any ideas?

I am trying to set an attribute’s (activated) date on an update action only when it changes from ‘I’ to ‘A’. Below is my current action that is not working. Any help with getting this right?


public function actionUpdate($id) {

	$model = $this->loadModel($id, 'Member');


	if (isset($_POST['Member'])) {

		$model->setAttributes($_POST['Member']);


		if ($model->activated == 'A') $model->activated = date('Y-m-d');


		if ($model->save()) {

			$this->redirect(array('view', 'id' => $model->Member));

		}

	}


	$this->render('update', array(

			'model' => $model,

));

}
2

You probably forgot to change Organization to Member




  if (isset($_POST['Member'])) {

/Tommy

3

Thanks Tommy, that is one of the problems but it still did not do the trick. Corrected original question.

4

Does the model have any errors? You could check by doing this:




if ($model->hasErrors()) {


   $errors = $model->getErrors();


   //then loop through the errors to see them...


}
5

Not coming up with any errors either, in my logs I can see that the sql being executed for activated is ‘0000-00-00’.

6

I supposed, within your logs you could see an UPDATE SQL command was dumped by yii? Copy and Paste it to phpmyadmin or chive or anything, then try to execute it, see if it is working manually to make sure…

7

Yeah, please paste the generated SQL query here. Is Y-m-d the format Mysql is expecting?

Also, why setAttributes and not just:

$model->attributes = $_POST[‘Member’]

Not sure it matters just curious :)

After you assign the date to the model field conditionally what does this show:

print_r($model);

8

Have you checked the value for $model->activated? It’s possible that it’s not getting set and failing the if statement.

9

Are you storing both the date Y-m-d and a text value ‘A’ in the same model field?

Are you sure you don’t have a separate status field and activated date field?

10

Thanks for the help everyone, found the stupid mistake…

I should have had [note the status == ‘A’]


if ($model->status == 'A') $model->activated = date('Y-m-d')
11

Thanks, I found this at the same time as you I guess, I was typing up a reply with the SQL generated with and without the if clause and finally found it. I appreciate the help.

and BTW the setAttributes is from giix generated code rather than gii