Sco\Admin\Traits\EntrustPermissionTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	36.84 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	roles()	0	7	1
A	boot()	14	14	2

<?php

namespace Sco\Admin\Traits;

/**
 * This file is part of Entrust,
 * a role & permission management solution for Laravel.
 *
 * @license MIT
 * @package Zizaco\Entrust
 */

use Illuminate\Support\Facades\Config;

trait EntrustPermissionTrait
{
    /**
     * Many-to-Many relations with role model.
     *
     * @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
     */
    public function roles()
    {
        return $this->belongsToMany(Config::get('admin.role'),

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

            Config::get('admin.permission_role_table'),
            Config::get('admin.permission_foreign_key'),
            Config::get('admin.role_foreign_key'));
    }

    /**
     * Boot the permission model
     * Attach event listener to remove the many-to-many records when trying to
     * delete Will NOT delete any records if the permission model uses soft
     * deletes.
     *
     * @return void|bool
     */
    public static function boot()

This method seems to be duplicated in your project.

    {
        parent::boot();

        static::deleting(function ($permission) {
            if (!method_exists(Config::get('admin.permission'),
                'bootSoftDeletes')
            ) {
                $permission->roles()->sync([]);
            }

            return true;
        });
    }
}