Email verification refactor

app/Events/EmailAddressWasChanged.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace App\Events;
+
+use App\User;
+use Illuminate\Foundation\Events\Dispatchable;
+use Illuminate\Queue\SerializesModels;
+
+class EmailAddressWasChanged
+{
+    use Dispatchable;
+    use SerializesModels;
+
+    /**
+     * @var \App\User
+     */
+    public $user;
+
+    public function __construct(User $user)
+    {
+        $this->user = $user;
+    }
+}

app/Http/Controllers/Articles/ArticlesController.php

@@ -4,22 +4,22 @@
 
 use App\Http\Controllers\Controller;
 use App\Http\Middleware\Authenticate;
-use App\Http\Middleware\RedirectIfUnconfirmed;
 use App\Http\Requests\ArticleRequest;
 use App\Jobs\CreateArticle;
 use App\Jobs\DeleteArticle;
 use App\Jobs\UpdateArticle;
 use App\Models\Article;
 use App\Models\Tag;
 use App\Policies\ArticlePolicy;
+use Illuminate\Auth\Middleware\EnsureEmailIsVerified;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Auth;
 
 class ArticlesController extends Controller
 {
     public function __construct()
     {
-        $this->middleware([Authenticate::class, RedirectIfUnconfirmed::class], ['except' => ['index', 'show']]);
+        $this->middleware([Authenticate::class, EnsureEmailIsVerified::class], ['except' => ['index', 'show']]);
     }
 
     public function index()

app/Http/Controllers/Articles/AuthoredArticles.php

@@ -4,14 +4,14 @@
 
 use App\Http\Controllers\Controller;
 use App\Http\Middleware\Authenticate;
-use App\Http\Middleware\RedirectIfUnconfirmed;
+use Illuminate\Auth\Middleware\EnsureEmailIsVerified;
 use Illuminate\Http\Request;
 
 class AuthoredArticles extends Controller
 {
     public function __construct()
     {
-        $this->middleware([Authenticate::class, RedirectIfUnconfirmed::class]);
+        $this->middleware([Authenticate::class, EnsureEmailIsVerified::class]);
     }
 
     public function __invoke(Request $request)

app/Http/Controllers/Articles/AuthoredSeries.php

@@ -4,14 +4,14 @@
 
 use App\Http\Controllers\Controller;
 use App\Http\Middleware\Authenticate;
-use App\Http\Middleware\RedirectIfUnconfirmed;
+use Illuminate\Auth\Middleware\EnsureEmailIsVerified;
 use Illuminate\Http\Request;
 
 class AuthoredSeries extends Controller
 {
     public function __construct()
     {
-        $this->middleware([Authenticate::class, RedirectIfUnconfirmed::class]);
+        $this->middleware([Authenticate::class, EnsureEmailIsVerified::class]);
     }
 
     public function __invoke(Request $request)

app/Http/Controllers/Articles/SeriesController.php

@@ -4,20 +4,20 @@
 
 use App\Http\Controllers\Controller;
 use App\Http\Middleware\Authenticate;
-use App\Http\Middleware\RedirectIfUnconfirmed;
 use App\Http\Requests\SeriesRequest;
 use App\Jobs\CreateSeries;
 use App\Jobs\DeleteSeries;
 use App\Jobs\UpdateSeries;
 use App\Models\Series;
 use App\Models\Tag;
 use App\Policies\SeriesPolicy;
+use Illuminate\Auth\Middleware\EnsureEmailIsVerified;
 
 class SeriesController extends Controller
 {
     public function __construct()
     {
-        $this->middleware([Authenticate::class, RedirectIfUnconfirmed::class], ['except' => ['index']]);
+        $this->middleware([Authenticate::class, EnsureEmailIsVerified::class], ['except' => ['index']]);
     }
 
     public function create()

app/Http/Controllers/Auth/LoginController.php

@@ -4,6 +4,7 @@
 
 use App\Http\Controllers\Controller;
 use App\Http\Middleware\RedirectIfAuthenticated;
+use App\Providers\RouteServiceProvider;
 use Illuminate\Foundation\Auth\AuthenticatesUsers;
 
 class LoginController extends Controller
@@ -26,7 +27,7 @@ class LoginController extends Controller
      *
      * @var string
      */
-    protected $redirectTo = '/dashboard';
+    protected $redirectTo = RouteServiceProvider::HOME;
 
     /**
      * Create a new controller instance.

app/Http/Controllers/Auth/RegisterController.php

@@ -6,7 +6,7 @@
 use App\Http\Middleware\RedirectIfAuthenticated;
 use App\Http\Requests\RegisterRequest;
 use App\Jobs\RegisterUser;
-use App\Jobs\SendEmailConfirmation;
+use App\Providers\RouteServiceProvider;
 use App\User;
 use Illuminate\Contracts\Validation\Validator as ValidatorContract;
 use Illuminate\Foundation\Auth\RegistersUsers;
@@ -32,7 +32,7 @@ class RegisterController extends Controller
      *
      * @var string
      */
-    protected $redirectTo = '/dashboard';
+    protected $redirectTo = RouteServiceProvider::HOME;
 
     /**
      * Create a new controller instance.
@@ -57,10 +57,6 @@ protected function validator(array $data): ValidatorContract
      */
     protected function create(array $data): User
     {
-        $user = $this->dispatchNow(RegisterUser::fromRequest(app(RegisterRequest::class)));
-
-        $this->dispatch(new SendEmailConfirmation($user));
-
-        return $user;
+        return $this->dispatchNow(RegisterUser::fromRequest(app(RegisterRequest::class)));
     }
 }

app/Http/Controllers/Auth/ResetPasswordController.php

@@ -4,6 +4,7 @@
 
 use App\Http\Controllers\Controller;
 use App\Http\Middleware\RedirectIfAuthenticated;
+use App\Providers\RouteServiceProvider;
 use Illuminate\Foundation\Auth\ResetsPasswords;
 
 class ResetPasswordController extends Controller
@@ -26,7 +27,7 @@ class ResetPasswordController extends Controller
      *
      * @var string
      */
-    protected $redirectTo = '/dashboard';
+    protected $redirectTo = RouteServiceProvider::HOME;
 
     /**
      * Create a new controller instance.

app/Http/Controllers/Auth/VerificationController.php

@@ -3,7 +3,9 @@
 namespace App\Http\Controllers\Auth;
 
 use App\Http\Controllers\Controller;
+use App\Providers\RouteServiceProvider;
 use Illuminate\Foundation\Auth\VerifiesEmails;
+use Illuminate\Http\Request;
 
 class VerificationController extends Controller
 {
@@ -14,18 +16,21 @@ class VerificationController extends Controller
     |
     | This controller is responsible for handling email verification for any
     | user that recently registered with the application. Emails may also
-    | be resent if the user did not receive the original email message.
+    | be re-sent if the user didn't receive the original email message.
     |
     */
 
-    use VerifiesEmails;
+    use VerifiesEmails {
+        resend as traitResend;
+        verify as traitVerify;
+    }
 
     /**
      * Where to redirect users after verification.
      *
      * @var string
      */
-    protected $redirectTo = '/home';
+    protected $redirectTo = RouteServiceProvider::HOME;
 
     /**
      * Create a new controller instance.
@@ -38,4 +43,32 @@ public function __construct()
         $this->middleware('signed')->only('verify');
         $this->middleware('throttle:6,1')->only('verify', 'resend');
     }
+
+    public function verify(Request $request)
+    {
+        /** @var \Illuminate\Http\RedirectResponse $response */
+        $response = $this->traitVerify($request);
+
+        if ($response->getSession()->has('verified')) {
+            $this->success('auth.verification.success');
+        } else {
+            $this->error('auth.verification.no_match');
+        }
+
+        return $response;
+    }
+
+    public function resend(Request $request)
+    {
+        /** @var \Illuminate\Http\RedirectResponse $response */
+        $response = $this->traitResend($request);
+
+        if ($response->getSession()->has('resent')) {
+            $this->success('auth.verification.sent', $request->user()->emailAddress());
+        } else {
+            $this->error('auth.verification.already_verified');
+        }
+
+        return $response;
+    }
 }

app/Http/Controllers/Forum/ThreadsController.php

@@ -3,7 +3,6 @@
 namespace App\Http\Controllers\Forum;
 
 use App\Http\Controllers\Controller;
-use App\Http\Middleware\RedirectIfUnconfirmed;
 use App\Http\Requests\ThreadRequest;
 use App\Jobs\CreateThread;
 use App\Jobs\DeleteThread;
@@ -18,13 +17,14 @@
 use App\Policies\ThreadPolicy;
 use App\Queries\SearchThreads;
 use Illuminate\Auth\Middleware\Authenticate;
+use Illuminate\Auth\Middleware\EnsureEmailIsVerified;
 use Illuminate\Http\Request;
 
 class ThreadsController extends Controller
 {
     public function __construct()
     {
-        $this->middleware([Authenticate::class, RedirectIfUnconfirmed::class], ['except' => ['overview', 'show']]);
+        $this->middleware([Authenticate::class, EnsureEmailIsVerified::class], ['except' => ['overview', 'show']]);
     }
 
     public function overview()

app/Http/Controllers/ReplyController.php

@@ -2,7 +2,6 @@
 
 namespace App\Http\Controllers;
 
-use App\Http\Middleware\RedirectIfUnconfirmed;
 use App\Http\Requests\CreateReplyRequest;
 use App\Http\Requests\UpdateReplyRequest;
 use App\Jobs\CreateReply;
@@ -13,13 +12,14 @@
 use App\Models\Thread;
 use App\Policies\ReplyPolicy;
 use Illuminate\Auth\Middleware\Authenticate;
+use Illuminate\Auth\Middleware\EnsureEmailIsVerified;
 use Illuminate\Http\RedirectResponse;
 
 class ReplyController extends Controller
 {
     public function __construct()
     {
-        $this->middleware([Authenticate::class, RedirectIfUnconfirmed::class]);
+        $this->middleware([Authenticate::class, EnsureEmailIsVerified::class]);
     }
 
     public function store(CreateReplyRequest $request)

app/Http/Requests/Request.php

@@ -10,12 +10,6 @@ abstract class Request extends FormRequest
 {
     use SendsAlerts;
 
-    public function authorize()
-    {
-        // Allow all requests and handle authorization in controllers.
-        return true;
-    }
-
     protected function failedValidation(Validator $validator)
     {
         $this->error('errors.fields');