Yajra\Acl\Traits\AuthorizesPermissionResources

Complexity

Total Complexity

5

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	authorizePermissionResource()	0	16	2
A	resourcePermissionMap()	0	8	3

<?php

namespace Yajra\Acl\Traits;

use Illuminate\Support\Collection;
use Illuminate\Support\Fluent;

/**
 * @mixin \Illuminate\Routing\Controller
 */
trait AuthorizesPermissionResources
{
    /**
     * Permission resource ability mapping.
     *
     * @var array
     */
    protected $resourcePermissionMap = [
        'index' => 'viewAny',
        'create' => 'create',
        'store' => 'create',
        'show' => 'view',
        'edit' => 'update',
        'update' => 'update',
        'destroy' => 'delete',
    ];

    /**
     * Authorize a permission resource action based on the incoming request.
     *
     * @param  string  $resource
     * @param  array  $options
     * @return void
     */
    public function authorizePermissionResource(string $resource, array $options = [])
    {
        $permissions = $this->resourcePermissionMap();
        $collection = new Collection;
        foreach ($permissions as $method => $ability) {
            $collection->push(new Fluent([
                'ability' => $ability,
                'method' => $method,
            ]));
        }

        $collection->groupBy('ability')->each(function ($permission, $ability) use ($resource, $options) {
            $this->middleware("can:{$resource}.{$ability}")

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

                ->only($permission->pluck('method')->toArray());
        });
    }

    /**
     * Get the map of permission resource methods to ability names.
     *
     * @return array
     */
    protected function resourcePermissionMap(): array
    {
        if (property_exists($this, 'customPermissionMap') && is_array($this->customPermissionMap)) {
            return array_merge($this->resourcePermissionMap, $this->customPermissionMap);

The property customPermissionMap does not exist. Did you maybe forget to declare it?

        }

        return $this->resourcePermissionMap;
    }
}