App\Modules\Roles\Repositories\RoleRepository

Complexity

Total Complexity

2

Size/Duplication

Total Lines	31
Duplicated Lines	32.26 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	assignPermissions()	10	10	1

<?php namespace App\Modules\Roles\Repositories;

use App\Modules\Core\BaseClasses\BaseRepository;
use App\Modules\Roles\Role;

class RoleRepository extends BaseRepository
{
    /**
     * Init new object.
     *
     * @param   Role $model
     * @return  void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(Role $model)
    {
        parent::__construct($model);
    }

    /**
     * Assign the given permission ids to the given role.
     *
     * @param  integer $roleId
     * @param  array   $permissionIds
     * @return object
     */
    public function assignPermissions($roleId, $permissionIds)

This method seems to be duplicated in your project.

    {
        \DB::transaction(function () use ($roleId, $permissionIds) {
            $role = $this->find($roleId);
            $role->permissions()->detach();
            $role->permissions()->attach($permissionIds);
        });

        return $this->find($roleId);
    }
}