jeremykenedy\LaravelRoles\Traits\RolesUsageAuthTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	3

<?php

namespace jeremykenedy\LaravelRoles\Traits;

trait RolesUsageAuthTrait
{
    /**
     * Variable to hold if we are using built in Laravel authentication.
     */
    private $_rolesGuiAuthEnabled;

    /**
     * Variable to hold if we are using roles/permissoins middleware for access.
     */
    private $_rolesGuiMiddlewareEnabled;

    /**
     * Variable to hold what roles/permissions middleware we are using if enabled.
     */
    private $_rolesGuiMiddleware;

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->_rolesGuiAuthEnabled = config('roles.rolesGuiAuthEnabled');
        $this->_rolesGuiMiddlewareEnabled = config('roles.rolesGuiMiddlewareEnabled');
        $this->_rolesGuiMiddleware = config('roles.rolesGuiMiddleware');

        if ($this->_rolesGuiAuthEnabled) {
            $this->middleware('auth');

It seems like middleware() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        if ($this->_rolesGuiMiddlewareEnabled) {
            $this->middleware($this->_rolesGuiMiddleware);
        }
    }
}