Terranet\Administrator\Traits\Actions\ActionSkeleton

Complexity

Total Complexity

8

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	authorize()	0	8	1
A	confirmationMessage()	0	3	1
A	hideFromIndex()	0	3	1
A	attributes()	0	8	2
A	route()	0	6	2
A	hideFromView()	0	3	1

<?php

namespace Terranet\Administrator\Traits\Actions;

use Illuminate\Database\Eloquent\Model;

The type Illuminate\Database\Eloquent\Model was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Foundation\Auth\User;

The type Illuminate\Foundation\Auth\User was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Support\Str;
use Terranet\Administrator\Contracts\Module;

trait ActionSkeleton
{
    /**
     * Check if specified user is authorized to execute this action.
     *
     * @param  User  $viewer
     * @param  Model  $model
     * @return bool
     */
    public function authorize(User $viewer, Model $model = null)

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

    {
        /** @var Module $resource */
        $resource = app('scaffold.module');

The function app was not found. Maybe you did not declare it correctly or list all dependencies?

        return $resource->actions()->authorize(
            Str::snake(class_basename($this)),
            $model
        );
    }

    /**
     * @param  Model  $model
     * @return string
     */
    public function route(Model $model = null)
    {
        return route('scaffold.action', [

The function route was not found. Maybe you did not declare it correctly or list all dependencies?

            'module' => app('scaffold.module'),

The function app was not found. Maybe you did not declare it correctly or list all dependencies?

            'id' => $model ? $model->getKey() : null,
            'action' => $this->action($model),

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

        ]);
    }

    /**
     * Confirmation message, if required.
     *
     * @return mixed null|string
     */
    protected function confirmationMessage(): ?string
    {
        return null;
    }

    /**
     * Hide an action from index page.
     *
     * @return bool
     */
    public function hideFromIndex(): bool
    {
        return false;
    }

    /**
     * Hide an action from view page.
     *
     * @return bool
     */
    public function hideFromView(): bool
    {
        return false;
    }

    /**
     * @param  Model  $model
     * @return string
     */
    protected function attributes(Model $model = null)

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

    {
        $attributes = [];
        if ($msg = $this->confirmationMessage()) {

Are you sure the assignment to $msg is correct as $this->confirmationMessage() targeting Terranet\Administrator\Traits\Actions\ActionSkeleton::confirmationMessage() seems to always return null.

            $attributes["onclick"] = "return window.confirm('{$msg}')";
        }

        return \admin\helpers\html_attributes($attributes);
    }
}