Skip to content

Commit 146a438

Browse files
committed
update methods to better handle ACTIVATION=false
1 parent 7c4c7f9 commit 146a438

File tree

3 files changed

+25
-5
lines changed

3 files changed

+25
-5
lines changed

app/Http/Controllers/Auth/ActivateController.php

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,15 +86,21 @@ public static function activeRedirect($user, $currentRoute)
8686
if ($user->activated) {
8787
Log::info('Activated user attempted to visit '.$currentRoute.'. ', [$user]);
8888

89+
$message = trans('auth.regThanks');
90+
if (config('settings.activation')) {
91+
$message = trans('auth.alreadyActivated');
92+
}
93+
8994
if ($user->isAdmin()) {
9095
return redirect()->route(self::getAdminHomeRoute())
9196
->with('status', 'info')
92-
->with('message', trans('auth.alreadyActivated'));
97+
->with('message', $message);
9398
}
9499

100+
95101
return redirect()->route(self::getUserHomeRoute())
96102
->with('status', 'info')
97-
->with('message', trans('auth.alreadyActivated'));
103+
->with('message', $message);
98104
}
99105

100106
return false;

app/Http/Controllers/Auth/RegisterController.php

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
namespace App\Http\Controllers\Auth;
44

55
use App\Http\Controllers\Controller;
6+
use App\Models\Profile;
67
use App\Models\User;
78
use App\Traits\ActivationTrait;
89
use App\Traits\CaptchaTrait;
@@ -101,7 +102,14 @@ protected function validator(array $data)
101102
protected function create(array $data)
102103
{
103104
$ipAddress = new CaptureIpTrait();
104-
$role = Role::where('slug', '=', 'unverified')->first();
105+
106+
if (config('settings.activation')) {
107+
$role = Role::where('slug', '=', 'unverified')->first();
108+
$activated = false;
109+
} else {
110+
$role = Role::where('slug', '=', 'user')->first();
111+
$activated = true;
112+
}
105113

106114
$user = User::create([
107115
'name' => $data['name'],
@@ -111,12 +119,18 @@ protected function create(array $data)
111119
'password' => Hash::make($data['password']),
112120
'token' => str_random(64),
113121
'signup_ip_address' => $ipAddress->getClientIp(),
114-
'activated' => ! config('settings.activation'),
122+
'activated' => $activated,
115123
]);
116124

117125
$user->attachRole($role);
118126
$this->initiateEmailActivation($user);
119127

128+
if (! config('settings.activation')) {
129+
$profile = new Profile();
130+
$user->profile()->save($profile);
131+
$user->save();
132+
}
133+
120134
return $user;
121135
}
122136
}

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
"intervention/image": "^2.5",
1919
"jaybizzle/laravel-crawler-detect": "^1.2",
2020
"jeremykenedy/laravel-blocker": "^2.2",
21-
"jeremykenedy/laravel-exception-notifier": "^1.2",
21+
"jeremykenedy/laravel-exception-notifier": "^2.1.0",
2222
"jeremykenedy/laravel-https": "^1.1",
2323
"jeremykenedy/laravel-logger": "^3.2",
2424
"jeremykenedy/laravel-phpinfo": "^1.2",

0 commit comments

Comments
 (0)