Merge branch '3.0'

Author: jrean

README.md

@@ -3,11 +3,21 @@ easily handle a user verification and validate the e-mail.
 
 [![Latest Stable Version](https://poser.pugx.org/jrean/laravel-user-verification/v/stable)](https://packagist.org/packages/jrean/laravel-user-verification) [![Total Downloads](https://poser.pugx.org/jrean/laravel-user-verification/downloads)](https://packagist.org/packages/jrean/laravel-user-verification) [![License](https://poser.pugx.org/jrean/laravel-user-verification/license)](https://packagist.org/packages/jrean/laravel-user-verification)
 
+## Versions
+**This package is Laravel 5.3 compliant.**
+
+For Laravel 5.3.*, use branch
+[3.0](https://github.com/jrean/laravel-user-verification/tree/3.0)
+and/or [master](https://github.com/jrean/laravel-user-verification/tree/master)
+
+For Laravel 5.0.* | 5.1.* | 5.2.*, use branch
+[2.2](https://github.com/jrean/laravel-user-verification/tree/2.2)
+
 ## About
 
 - [x] Generate and store a verification token for a registered user
 - [x] Send an e-mail with the verification token link
-- [x] Handle the verification of the token
+- [x] Handle the token verification
 - [x] Set the user as verified
 - [x] Relaunch the process anytime
 
@@ -19,7 +29,7 @@ require block of your composer.json file:
 
     {
         "require": {
-                "jrean/laravel-user-verification": "^2.0"
+                "jrean/laravel-user-verification": "^3.0"
         }
 
     }
@@ -48,7 +58,7 @@ Open up `config/app.php` and add the following to the `aliases` key:
 
 ## Configuration
 
-Prior to use this package the table representing the user must be updated with
+Prior to use this package, the table representing the user must be updated with
 two new columns, `verified` and `verification_token`.
 
 **It is mandatory to add the two columns on the same table and where the user's
@@ -68,7 +78,7 @@ php artisan make:migration add_verification_to_:table_table --table=":table"
 
 Where `:table` is replaced by the table name of your choice.
 
-For instance if you want to keep the default Eloquent User table:
+For instance, if you want to keep the default Eloquent `User` table:
 
 ```
 php artisan make:migration add_verification_to_users_table --table="users"
@@ -114,13 +124,6 @@ Migrate the migration with the following command:
 php artisan migrate
 ```
 
-### Exception
-
-If the table representing the user is not updated with the two new columns and
-the model representing the user doesn't implement the authenticatable interface
-`Illuminate\Contracts\Auth\Authenticatable`, a `ModelNotCompliantException`
-will be thrown.
-
 ## E-mail
 
 This package provides a method to send an e-mail with a link containing the verification token.
@@ -162,7 +165,7 @@ Click here to verify your account: <a href="{{ $link = url('verification', $user
 ## Errors
 
 This package throws several exceptions. You are free to use `try/catch`
-statements or to rely on the Laravel built-in exceptions handling.
+statements or to rely on the Laravel built-in exceptions handler.
 
 * `ModelNotCompliantException`
 
@@ -196,12 +199,12 @@ verify, ...).
 
 ### Routes
 
-Add the two (2) default routes to the `app\Http\routes.php` file. Routes are
+Add the two (2) default routes to the `routes/web.php` file. Routes are
 customizable.
 
 ```
-    Route::get('verification/error', 'Auth\AuthController@getVerificationError');
-    Route::get('verification/{token}', 'Auth\AuthController@getVerification');
+    Route::get('verification/error', 'Auth\RegisterController@getVerificationError');
+    Route::get('verification/{token}', 'Auth\RegisterController@getVerification');
 ```
 
 ### Trait
@@ -289,7 +292,7 @@ package as well as created and migrated the migration according to this
 documentation and the previous documented steps.**
 
 Note that by default the behaviour of Laravel is to return an authenticated
-user straight after the registration step.
+user after the registration step.
 
 ### Example
 
@@ -298,29 +301,29 @@ following Laravel logic. You are free to implement the way you want.
 It is highly recommended to read and to understand the way Laravel implements
 registration/authentication.
 
-Edit the `app\Http\routes.php` file.
+Edit the `routes/web.php` file.
 
 - Define two (2) new routes.
 
 ```
-    Route::get('verification/error', 'Auth\AuthController@getVerificationError');
-    Route::get('verification/{token}', 'Auth\AuthController@getVerification');
+    Route::get('verification/error', 'Auth\RegisterController@getVerificationError');
+    Route::get('verification/{token}', 'Auth\RegisterController@getVerification');
 ```
 
 - Define the e-mail view.
 
-Edit the `app\Http\Controllers\Auth\AuthController.php` file.
+Edit the `app\Http\Controllers\Auth\RegisterController.php` file.
 
 - [x] Import the `VerifiesUsers` trait (mandatory)
 - [ ] Overwrite and customize the redirect attributes/properties paths
     available within the `RedirectsUsers` trait included by the
     `VerifiesUsers` trait. (not mandatory)
-- [ ] Overwrite the error view name used by the `getVerificationError()` method
+- [ ] Overwrite the default error view name used by the `getVerificationError()` method
     (not mandatory)
-- [x] Create the verification error view (mandatory)
+- [x] Create the verification error view at
+    `resources/views/errors/user-verification.blade.php` (mandatory)
 - [ ] Overwrite the contructor (not mandatory)
-- [x] Overwrite the `postRegister()`/`register()` method depending on the
-    Laravel version you use (mandatory)
+- [x] Overwrite the `register()` method (mandatory)
 
 ```
 
@@ -329,78 +332,72 @@ Edit the `app\Http\Controllers\Auth\AuthController.php` file.
     use App\User;
     use Validator;
     use App\Http\Controllers\Controller;
-    use Illuminate\Foundation\Auth\ThrottlesLogins;
-    use Illuminate\Foundation\Auth\AuthenticatesAndRegistersUsers;
+    use Illuminate\Foundation\Auth\RegistersUsers;
+    use Illuminate\Http\Request;
+
     use Jrean\UserVerification\Traits\VerifiesUsers;
     use Jrean\UserVerification\Facades\UserVerification;
-    use Illuminate\Http\Request;
-    use Illuminate\Support\Facades\Auth;
 
-    class AuthController extends Controller
+
+    class RegisterController extends Controller
     {
         /*
         |--------------------------------------------------------------------------
-        | Registration & Login Controller
+        | Register Controller
         |--------------------------------------------------------------------------
         |
-        | This controller handles the registration of new users, as well as the
-        | authentication of existing users. By default, this controller uses
-        | a simple trait to add these behaviors. Why don't you explore it?
+        | This controller handles the registration of new users as well as their
+        | validation and creation. By default this controller uses a trait to
+        | provide this functionality without requiring any additional code.
         |
         */
 
-        use AuthenticatesAndRegistersUsers, ThrottlesLogins;
+        use RegistersUsers;
 
         use VerifiesUsers;
 
-        /**
-        * Create a new authentication controller instance.
+       /**
+        * Create a new controller instance.
         *
         * @return void
         */
         public function __construct()
         {
-            // Based on the workflow you want you may update and customize the following lines.
-
-            // Laravel 5.0.*|5.1.*
-            $this->middleware('guest', ['except' => ['getLogout', 'getVerification', 'getVerificationError']]);
+            // Based on the workflow you need, you may update and customize the following lines.
 
-            // Laravel 5.2.*
-            $this->middleware('guest', ['except' => ['logout', 'getVerification, 'getVerificationError]]);
-            //or
-            $this->middleware($this->guestMiddleware(), ['except' => ['logout', 'getVerification', 'getVerificationError]]);
+            $this->middleware('guest', ['except' => ['getVerification', 'getVerificationError']]);
         }
 
-        // Laravel 5.0.*|5.1.*
         /**
-        * Handle a registration request for the application.
+        * Get a validator for an incoming registration request.
         *
-        * @param  \Illuminate\Http\Request  $request
-        * @return \Illuminate\Http\Response
+        * @param  array  $data
+        * @return \Illuminate\Contracts\Validation\Validator
         */
-        public function postRegister(Request $request)
+        protected function validator(array $data)
         {
-            $validator = $this->validator($request->all());
-
-            if ($validator->fails()) {
-                $this->throwValidationException(
-                    $request, $validator
-                );
-            }
-
-            $user = $this->create($request->all());
-
-            // Authenticating the user is not mandatory at all.
-            Auth::login($user);
-
-            UserVerification::generate($user);
-
-            UserVerification::send($user, 'My Custom E-mail Subject');
+            return Validator::make($data, [
+                'name' => 'required|max:255',
+                'email' => 'required|email|max:255|unique:users',
+                'password' => 'required|min:6|confirmed',
+            ]);
+        }
 
-            return redirect($this->redirectPath());
+        /**
+        * Create a new user instance after a valid registration.
+        *
+        * @param  array  $data
+        * @return User
+        */
+        protected function create(array $data)
+        {
+            return User::create([
+                'name' => $data['name'],
+                'email' => $data['email'],
+                'password' => bcrypt($data['password']),
+            ]);
         }
 
-        // Laravel 5.2.*
         /**
         * Handle a registration request for the application.
         *
@@ -409,26 +406,12 @@ Edit the `app\Http\Controllers\Auth\AuthController.php` file.
         */
         public function register(Request $request)
         {
-            $validator = $this->validator($request->all());
-
-            if ($validator->fails()) {
-                $this->throwValidationException(
-                    $request, $validator
-                );
-            }
+            $this->validator($request->all())->validate();
 
             $user = $this->create($request->all());
-
-            // Authenticating the user is not mandatory at all.
-
-            // Laravel <= 5.2.7
-            // Auth::login($user);
-
-            // Laravel > 5.2.7
-            Auth::guard($this->getGuard())->login($user);
+            $this->guard()->login($user);
 
             UserVerification::generate($user);
-
             UserVerification::send($user, 'My Custom E-mail Subject');
 
             return redirect($this->redirectPath());
@@ -446,7 +429,7 @@ update the middleware exception to allow `getVerification` and
 `getVerificationError` routes to be accessed.
 
 ```
-$this->middleware($this->guestMiddleware(), ['except' => ['logout', 'getVerification', 'getVerificationError']]);
+$this->middleware('guest', ['except' => ['getVerification', 'getVerificationError']]);
 ```
 
 ## Contribute

composer.json

@@ -1,7 +1,7 @@
 {
     "name": "jrean/laravel-user-verification",
     "description": "User Email Verification For Laravel 5.*",
-    "keywords": ["laravel", "framework", "user verification", "email verification"],
+    "keywords": ["laravel", "framework", "user verification", "email verification", "user activation", "email activation"],
     "license": "MIT",
     "authors": [
         {
@@ -10,22 +10,14 @@
         }
     ],
     "require": {
-        "php": ">=5.5.9",
-        "illuminate/support": "5.0.*|5.1.*|5.2.*"
-    },
-    "require-dev": {
-        "phpunit/phpunit": "^4.7.6"
+        "php": ">=5.6.4",
+        "illuminate/support": "5.3.*"
     },
     "autoload": {
         "psr-4": {
             "Jrean\\UserVerification\\": "src/"
         }
     },
-    "autoload-dev": {
-        "psr-4": {
-            "Jrean\\Tests\\UserVerification\\": "tests/"
-        }
-    },
     "config": {
         "preferred-install": "dist"
     },