Question

When I delete a ‘type’ i set isActive = 0;

Every ‘type’ ‘has many’ ‘causal’

So when disabling a type i want disable every causal

in type controller i’m trying this

$model = $this->loadModel($id);
$model->isActive = 0;

foreach ($model->causalsObj as $key => $causal ) {
   $causal = CausalController::delete($causal->id);
}

$model->save();

This doesn’t work (PHP error during ajax call)

Answer 1

That should go into the model, not the controller, I’d use afterSave. so in the CasualType:

public function afterSave(){
   if(!$this->isActive){
       Casual::model()->deleteAll('type_id = '.$this->id);
   }

   return parent::afterSave();
}

If you don’t actually mean ‘delete’ but deactivate you can still do this in one single query using CActiveRecord::updateAll:

public function afterSave(){
   if(!$this->isActive){
       Casual::model()->updateAll(array('isActive' => 0), 'type_id = '.$this->id);
   }

   return parent::afterSave();
}

Instantiating a controller in another controller does not make sense, controllers are there to handle user requests, not to hold your business logic