Afrittella\BackProject\Repositories\Permissions

Complexity

Total Complexity

13

Size/Duplication

Total Lines	92
Duplicated Lines	53.26 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	model()	0	4	1
A	create()	0	10	2
A	update()	8	8	2
A	firstOrCreate()	0	15	3
B	transform()	41	41	5

<?php
namespace Afrittella\BackProject\Repositories;

class Permissions extends Base
{
    public function model()
    {
        return config('laravel-permission.models.permission');
    }

    public function create(array $data)
    {
        $permission = $this->model->create($data);

The method create() does not exist on Illuminate\Database\Eloquent\Model. Did you maybe mean created()?

        if (!empty($data['roles'])) {
            $permission->roles()->sync($data['roles']);
        }

        return $permission;
    }

    public function update(array $data, $id, $attribute = 'id')

This method seems to be duplicated in your project.

    {
        $model_data = $this->findBy($attribute, $id);
        if (!empty($data['roles'])) {
            $model_data->roles()->sync($data['roles']);
        }
        return $model_data->update($data);
    }

    public function firstOrCreate($data)
    {
        // If permissions are in format permission1,permission2...
        if (!is_array($data)) {
            $perms = explode(',', $data);
        } else {
            $perms = $data;
        }

        foreach ($perms as $perm):
            $this->model->firstOrCreate([
                'name' => trim(strtolower($perm))
            ]);
        endforeach;
    }

    /**
     * Transform data in a table array for view
     * @param $data
     * @param array $options
     * @return array
     */
    public function transform($data = [], $options = [])

The parameter $options is not used and could be removed.

This method seems to be duplicated in your project.

    {
        if (empty($data)) {
            $data = $this->all();
        }

        // Table header
        $head = [
            'columns' => [
                trans('back-project::permissions.name'),
                trans('back-project::permissions.roles'),
                trans('back-project::crud.actions'),
            ]
        ];

        $body = [];

        foreach ($data as $row):
            $actions = [];

            if ($row->name !== 'administration' and $row->name !== 'backend') {

Using logical operators such as and instead of && is generally not recommended.

                $actions = [
                    'edit' => ['url' => route('bp.permissions.edit', [$row['id']])],
                    'delete' => ['url' => route('bp.permissions.delete', [$row['id']])]
                ];
            }

            $body[] = [
                'columns' => [
                    ['content' => $row->name],
                    ['content' => implode('<br>', $row->roles->pluck('name')->toArray())],
                    ['content' => false, 'actions' => $actions],
                ]
            ];
        endforeach;

        return [
            'head' => $head,
            'body' => $body
        ];
    }

}