Request validation + user email check

Author: jrean

README.md

@@ -214,6 +214,10 @@ The given user is not yet verified.
 
 No user found for the given e-mail adresse.
 
+* `UserHasNoEmailException`
+
+User email property is null or empty.
+
 ### Error View
 
 By default the `user-verification.blade.php` view will be loaded for the verification error route `/email-verification/error`. If an error occurs, the user will be redirected to this route and this view will be rendered.

src/Exceptions/UserHasNoEmailException.php

@@ -0,0 +1,19 @@
+<?php
+/**
+ * This file is part of Jrean\UserVerification package.
+ *
+ * (c) Jean Ragouin <[email protected]> <www.askjong.com>
+ */
+namespace Jrean\UserVerification\Exceptions;
+
+use Exception;
+
+class UserHasNoEmailException extends Exception
+{
+    /**
+     * The exception description.
+     *
+     * @var string
+     */
+    protected $message = 'The given user instance has an empty or null email field.';
+}

src/Traits/VerifiesUsers.php

@@ -25,7 +25,9 @@ trait VerifiesUsers
      */
     public function getVerification(Request $request, $token)
     {
-        $this->validateRequest($request);
+        if (! $this->validateRequest($request)) {
+            return redirect($this->redirectIfVerificationFails());
+        }
 
         try {
             UserVerificationFacade::process($request->input('email'), $token, $this->userTable());
@@ -54,7 +56,7 @@ public function getVerificationError()
      * Validate the verification link.
      *
      * @param  string  $token
-     * @return Response
+     * @return bool
      */
     protected function validateRequest(Request $request)
     {
@@ -63,7 +65,7 @@ protected function validateRequest(Request $request)
         ]);
 
         if ($validator->fails()) {
-            return redirect($this->redirectIfVerificationFails());
+            return false;
         }
     }
 

src/UserVerification.php

@@ -7,18 +7,18 @@
 namespace Jrean\UserVerification;
 
 use Illuminate\Contracts\Auth\Authenticatable as AuthenticatableContract;
-/* use Illuminate\Contracts\Mail\Mailer as MailerContract; */
-use Illuminate\Mail\Mailer;
 use Illuminate\Database\Schema\Builder;
+use Illuminate\Mail\Mailer;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Str;
-use Jrean\UserVerification\Mail\VerificationTokenGenerated;
 use Jrean\UserVerification\Events\UserVerified;
 use Jrean\UserVerification\Events\VerificationEmailSent;
 use Jrean\UserVerification\Exceptions\ModelNotCompliantException;
 use Jrean\UserVerification\Exceptions\TokenMismatchException;
+use Jrean\UserVerification\Exceptions\UserHasNoEmailException;
 use Jrean\UserVerification\Exceptions\UserIsVerifiedException;
 use Jrean\UserVerification\Exceptions\UserNotFoundException;
+use Jrean\UserVerification\Mail\VerificationTokenGenerated;
 
 class UserVerification
 {
@@ -57,6 +57,10 @@ public function __construct(Mailer $mailer, Builder $schema)
      */
     public function generate(AuthenticatableContract $user)
     {
+        if (empty($user->email)) {
+            throw new UserHasNoEmailException();
+        }
+
         return $this->saveToken($user, $this->generateToken());
     }