Arcanesoft\Auth\ViewComposers\RolesComposer

Complexity

Total Complexity

3

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	composeFilters()	0	19	2

<?php namespace Arcanesoft\Auth\ViewComposers;

use Arcanesoft\Auth\Bases\ViewComposer;
use Arcanesoft\Contracts\Auth\Models\Role;
use Illuminate\Contracts\View\View;

/**
 * Class     RolesComposer
 *
 * @package  Arcanesoft\Auth\ViewComposers
 * @author   ARCANEDEV <arcanedev.maroc@gmail.com>
 */
class RolesComposer extends ViewComposer
{
    /* ------------------------------------------------------------------------------------------------
     |  Properties
     | ------------------------------------------------------------------------------------------------
     */
    /**
     * The role model.
     *
     * @var \Arcanesoft\Contracts\Auth\Models\Role
     */
    private $role;

    /* ------------------------------------------------------------------------------------------------
     |  Constructor
     | ------------------------------------------------------------------------------------------------
     */
    /**
     * RolesComposer constructor.
     *
     * @param Role $role
     */
    public function __construct(Role $role)
    {
        $this->role = $role;
    }

    /* ------------------------------------------------------------------------------------------------
     |  Main Functions
     | ------------------------------------------------------------------------------------------------
     */
    public function composeFilters(View $view)
    {
        $filters   = [];

        $filters['all'] = link_to_route('auth::foundation.users.index', 'All');

        $roles   = $this->cacheResults('auth::roles.filters', function () {
            return $this->role->has('permissions')->get();

The method has() does not seem to exist on object<Arcanesoft\Contracts\Auth\Models\Role>.

        });

        foreach ($roles as $role) {
            /** @var  Role  $role */
            $filters[$role->slug] = link_to_route('auth::foundation.users.roles-filter.index', $role->name, [

Accessing slug on the interface Arcanesoft\Contracts\Auth\Models\Role suggest that you code against a concrete implementation. How about adding an instanceof check?

Accessing name on the interface Arcanesoft\Contracts\Auth\Models\Role suggest that you code against a concrete implementation. How about adding an instanceof check?

                $role->hashed_id

Accessing hashed_id on the interface Arcanesoft\Contracts\Auth\Models\Role suggest that you code against a concrete implementation. How about adding an instanceof check?

            ]);
        }

        $view->with('rolesFilters', $filters);
    }
}