This should be:

function activate($user_id = null, $in_hash = null) {
$this->User->id = $user_id;

Made a mod that should allow the activation code an additional day before it expires:

in app/controllers/users_controller:

function register() {
if (!empty($this->data)) {
$this->User->create();
$this->User->save($this->data);
$this->Session->setFlash(‘Your account has not been activated yet!’);
$this->__sendActivationEmail($this->User->getLastInsertID());
$this->redirect(‘/line’);
}
}

function __sendActivationEmail($user_id) {
$user = $this->User->find(array(‘User.id’ => $user_id), array(‘User.id’, ‘User.email’, ‘User.username’), null, false);
if ($user === false) {
debug(__METHOD__.” failed to retrieve User data for user.id: {$user_id}”);
return false;
}
// Set data for the “view” of the Email
$this->set(‘activate_url’, ‘http://’ . env(‘SERVER_NAME’) . ‘/users/activate/’ . $user['User']['id'] . ‘/’ . $this->User->getActivationHash());
$this->set(‘username’, $this->data['User']['username']);
$this->Email->to = $user['User']['email'];
$this->Email->subject = env(‘SERVER_NAME’) . ‘ – Please confirm your email address’;
$this->Email->from = ‘noreply@’ . env(‘SERVER_NAME’);
$this->Email->template = ‘user_confirm’;
$this->Email->sendAs = ‘text’; // ‘text’, ‘html’ and ‘both’ are options
return $this->Email->send();
}

function activate($user_id = null, $in_hash = null) {
$this->User->find(array(‘User.id’ => $this->params['pass'][0]), array(‘User.id’, ‘User.email’, ‘User.username’), null, false);
if ($this->User->getActivationHash() == $this->params['pass'][1] || $this->User->getYesterdaysActivationHash() == $this->params['pass'][1])
{
// Update the active flag in the database
$this->User->saveField(‘active’, 1);

// Let the user know they can now log in!
$this->Session->setFlash(‘Your account has been activated, please log in below’);
$this->redirect(‘login’);
}
// Activation failed, render ‘/views/user/activate.ctp’ which should tell the user.
}

… and in app/models/user.php

function getActivationHash()
{
if (!isset($this->id)) {
return false;
}
return substr(Security::hash(Configure::read(‘Security.salt’) . $this->field(‘created’) . date(‘y’) . date(‘m’). date(‘d’)), 0, 8);
}

function getYesterdaysActivationHash()
{
if (!isset($this->id)) {
return false;
}
return substr(Security::hash(Configure::read(‘Security.salt’) . $this->field(‘created’) . date(‘y’) . date(‘m’). date(‘d’) – 1), 0, 8);
}

Missing Component File

Error: The component file was not found.

Error: Create the class ‘Auth’Component in file: appcontrollerscomponents‘auth’.php

Notice: If you want to customize this error message, create appviewserrorsmissing_component_file.ctp

not getting wt is the proble…

file:// /app/controllers/users_controller.php

Add following line in the UsersController::beforeFilter()
{
…
$this->Auth->autoRedirect = false;
…
}
This will allow you to use further checking on login() action.

Good Job!!

function activate($user_id = null, $in_hash = null) {
$this->User->id = $id;

should actually be

function activate($user_id = null, $in_hash = null) {
$this->User->id = $user_id;

He referred to $id instead of $user_id, which was the real name of the variable passed to the function via the url, which is why this wouldn’t work/would give you errors if you just copied and pasted the above code. Just goes to show that using someone’s code without really knowing what it’s doing and how is a bad idea.

Cheers!

__sendActivationEmail(…)
$user['User']['id']
giving me Undefined index: id

function activate(…)
$this->User->id = $id;
giving me Undefined variable: id

Any chance someone can port this??

I hope this isn’t a stupid question – Sorry in advance if it is!