CakePHP: BeforeSave & BeforeValidate with PasswordableBehavior

Question

I'm trying to use DerEuromark's Passwordable behavior with my CakePHP app but am having trouble getting it to work. I followed the installation instructions (http://www.dereuromark.de/2011/08/25/working-with-passwords-in-cakephp/), modifying my controller and views, but I keep getting an error saying that my BeforeValidate and BeforeSave aren't compatible with the behavior - and of course, the behavior doesn't work.

I know I need to get these two set up correctly in my model, but I don't know what they should look like- the instructions didn't cover this point.

What would a basic, vanilla BeforeValidate and BeforeSave need to look like to work with this behavior?

Under my Users controller:

public function register() {
if ($this->request->is('post') || $this->request->is('put')) {
    $this->User->Behaviors->attach('Tools.Passwordable');
    if ($this->User->save($this->request->data, true, array('username', 'name', 'email', 'pwd', 'pwd_repeat', 'group_id'))) {
    $this->Session->setFlash(__('The user has been saved'), 'flash/success');
            $this->redirect(array('action' => 'index'));
} else {
            $this->Session->setFlash(__('The user could not be saved. Please, try again.'), 'flash/error');
        }
   unset($this->request->data['User']['pwd']);
    unset($this->request->data['User']['pwd_repeat']);
}

The Passwordable behavior with the BeforeValidate and BeforeSave that my user.php needs to be compatible with: https://github.com/dereuromark/tools/blob/master/Model/Behavior/PasswordableBehavior.php

The error:

Strict (2048): Declaration of PasswordableBehavior::beforeValidate() should be compatible with ModelBehavior::beforeValidate(Model $model, $options = Array) [APP/Plugin/Tools/Model/Behavior/PasswordableBehavior.php, line 338]
Strict (2048): Declaration of PasswordableBehavior::beforeSave() should be compatible with ModelBehavior::beforeSave(Model $model, $options = Array) [APP/Plugin/Tools/Model/Behavior/PasswordableBehavior.php, line 338]

Edit: User Model:

<?php
App::uses('AppModel', 'Model');
App::uses('AuthComponent', 'Controller/Component');
App::uses('PasswordableBehavior', 'Tools.Model/Behavior');
/**
 * User Model
 *
 * @property Group $Group
 * @property Post $Post
 */
class User extends AppModel {

    //simplified per-group only permissions- tell ACL to skip checking user AROs and only check group AROs
    public function bindNode($user) {
    return array('model' => 'Group', 'foreign_key' => $user['User']['group_id']);
}

/**
 * Validation rules
 *
 * @var array
 */
public $validate = array(
    'username' => array(
        'notEmpty' => array(
            'rule' => array('notEmpty'),
            //'message' => 'Your custom message here',
            //'allowEmpty' => false,
            //'required' => false,
                //'last' => false, // Stop validation after this rule
                //'on' => 'create', // Limit validation to 'create' or 'update' operations
            ),

                    'username' => array(
            'rule' => 'isUnique',
            'required' => true,
            'allowEmpty' => false,
            'on' => 'create',
            'last' => false,
            'message' => 'That username has already been taken'
    ),
        ),


        'email' => array(
            'email' => array(
                'rule' => array('email'),
                //'message' => 'Your custom message here',
                //'allowEmpty' => false,
                //'required' => false,
                //'last' => false, // Stop validation after this rule
                //'on' => 'create', // Limit validation to 'create' or 'update' operations
            ),
        ),
        'password' => array(
            'notEmpty' => array(
                'rule' => array('notEmpty'),
                //'message' => 'Your custom message here',
                //'allowEmpty' => false,
                //'required' => false,
                //'last' => false, // Stop validation after this rule
                //'on' => 'create', // Limit validation to 'create' or     'update' operations
            ),
        ),
        'group_id' => array(
            'numeric' => array(
                'rule' => array('numeric'),
                //'message' => 'Your custom message here',
                //'allowEmpty' => false,
                //'required' => false,
                //'last' => false, // Stop validation after this rule
            //'on' => 'create', // Limit validation to 'create' or 'update' operations
        ),
    ),
);

//The Associations below have been created with all possible keys, those that are         not needed can be removed

/**
 * belongsTo associations
 *
 * @var array
 */    
    public $belongsTo = array(
        'Group' => array(
            'className' => 'Group',
            'foreignKey' => 'group_id',
            'conditions' => '',
            'fields' => '',
            'order' => ''
        )
    );

    public $actsAs = array('Acl' => array('type' => 'requester'));

    public function parentNode() {
        if (!$this->id && empty($this->data)) {
            return null;
        }
        if (isset($this->data['User']['group_id'])) {
            $groupId = $this->data['User']['group_id'];
        } else {
            $groupId = $this->field('group_id');
        }
        if (!$groupId) {
            return null;
        } else {
            return array('Group' => array('id' => $groupId));
        }
    }

/**
 * hasMany associations
 *
 * @var array
 */
    public $hasMany = array(
        'Post' => array(
            'className' => 'Post',
            'foreignKey' => 'user_id',
            'dependent' => false,
            'conditions' => '',
            'fields' => '',
            'order' => '',
            'limit' => '',
            'offset' => '',
            'exclusive' => '',
            'finderQuery' => '',
            'counterQuery' => ''
        )
    );

    public function beforeValidate($options = array()) {

    }   


public function beforeSave($options = array()) {

} 

}

Answer 1

The error already tells you already pretty precisely what you have to do: Make the method signature of the behavior match the signature of the ModelBehavior methods signature.

Looks like the plugin was not updated for reflect the changes in the latest CakePHP releases. The options array was added.

Answer 2

Be sure that the method signature matches. Parameters of the method have to be the same as the parameters of the parent class' method.

You probably forgot them in your User model.

If you are really using the code you posted the link to, the method signatures are correct. Maybe you are using an older version or have a mistake in your User model.

You also have to update the $actsAs property in order to use the behavior.

public $actsAs = array(
    'Acl' => array('type' => 'requester'),
    'Passwordable',
);

Also, beforeSave and beforeValidate must return true in order to proceed to the saving process. Otherwise, it aborts. See http://book.cakephp.org/2.0/en/models/callback-methods.html.

Answer 3

in you PasswordableBehavior do this for both your methods (beforeValidate , beforeSave )

public function beforeValidate ($options=array()) {
   ...
   ...
}

//or try this if it didn't work
public function beforeValidate (Model $model, $options=array()) {
   ...
   ...
}