Question

I find my edit actions in CakePHP controllers get messy pretty quickly, and I’d like to pull most of that crap into the Model. Let me give you a scenario.

I have an users/edit action in my Users controller. I want to let users reset a password (or not reset the password) in my form. If they provide a new password then I pass the three password fields into save() using the fields list parameter of save(). If they don’t provide those fields I don’t want to pass those fields in using the fields list.

The code to check these fields is currently in my controller, what would be a good way to move this into the model?

Here’s what my controller’s edit action looks like:

function edit($id = null) {
    if ($this->Session->check('Auth.User') && $this->Session->read('Auth.User.id') == $id) {
        if (!$id && empty($this->data)) {
            $this->Session->setFlash('Invalid Account','default',array('class'=>'flash_error'));
            $this->redirect(array('controller'=>'directories', 'action' => 'index'));
        }
        if (!empty($this->data)) {
            // take out the following and an error occurs in parentNode()
            $this->data['User']['group_id'] = 2;
            if (empty($this->data['User']['old_password'])) {   //TODO: pass in a field list for every publicly available save() call.
                //dont update the password fields if they aren't passing in the old password
                if ($this->User->save($this->data,true,array('first_name', 'last_name', 'email', 'username'))) {
                    $this->Session->setFlash('Your changes have been saved','default',array('class'=>'flash_ok'));
                    $this->redirect(array('controller'=>'directories','action'=>'index'));
                } else {
                    $this->Session->setFlash('Your changes could not be saved. Please, try again.','default',array('class'=>'flash_error'));
                }                   
            } else {
                //update the passwords
                if ($this->User->save($this->data,true,array('first_name', 'last_name', 'email', 'username', 'password', 'password_confirm', 'old_password'))) {
                    $this->Session->setFlash('Your changes have been saved','default',array('class'=>'flash_ok'));
                    $this->redirect(array('controller'=>'directories','action'=>'index'));
                } else {
                    $this->Session->setFlash('Your changes could not be saved. Please, try again.','default',array('class'=>'flash_error'));
                }
            }
        }
        if (empty($this->data)) {
            $this->data = $this->User->read(array(
                'first_name', 'last_name', 'email', 'username'
            ), $id);
        }
        $this->set('user_id',$id);
        $this->set('current_subscription', $this->User->Subscription->currentSubscription($id));
    } else {
        //redirect to not authorized
        $this->Session->setFlash('Invalid Account','default',array('class'=>'flash_error'));
        $this->redirect(array('controller'=>'directories', 'action' => 'index'));
    }
}

Answer 1

A more graceful style would be

function edit($id = null)
{
     if($id && $this->Modelname->isValidLoginUser($id) && $this->data)
     {
         $login_tag = $this->Modelname->resetPass($id,$this->data);
         switch($login_tag)
         {
             case 0: $this->Session->setFlash();$this->redirect();break;
             case 1: $this->Session->setFlash();$this->redirect();break;
             ....
         }
     }
     else
     {
         $this->Session->setFlash("missing arguments.");
     }
}

And the function Modelname->resetPass() in the model looks like

function resetPass($id,$data)
{
     $user = $this->findById($id);
     $oldpasswd = $user[modelname]['password'];
     $newpasswd = $data[modelname][passwd1];
     $confirmpasswd = $data[modelname][passwd2];
     if($newpasswd=="" || $confirmpasswd=="")
     {
        return 0;
     }
     if($newpasswd != confirmpasswd)
     {
        return 1;
     }
     ....//perhaps other invalid situations

     if($newpasswd == $oldpasswd)
     {
       $this->saveField("password",$newpasswd);
       return N; //N is an int meaning success.
     }

}