FaithGen\SDK\Http\Middleware\ActivatedMinistryMiddleware

Complexity

Total Complexity

2

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	6	2

<?php

namespace FaithGen\SDK\Http\Middleware;

use Closure;

class ActivatedMinistryMiddleware
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if (auth(config('faithgen-sdk.guard'))->user()->activation->active) {

Accessing activation on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            return $next($request);
        } else {
            abort(403, 'You need to activate you account first to access this part');
        }
    }
}