Yaro\Jarboe\Http\Controllers\Traits\Handlers\CreateHandlerTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	55
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

86.67%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	handleCreate()	19	19	3
A	getCreateViewsAbove()	4	4	1
A	getCreateViewsBelow()	4	4	1
	init()	1	1	?
	bound()	1	1	?
	crud()	1	1	?
	can()	1	1	?

<?php

namespace Yaro\Jarboe\Http\Controllers\Traits\Handlers;

use Illuminate\Http\Request;
use Spatie\Permission\Exceptions\UnauthorizedException;
use Yaro\Jarboe\Exceptions\PermissionDenied;
use Yaro\Jarboe\Table\CRUD;

trait CreateHandlerTrait

This class seems to be duplicated in your project.

{
    /**
     * Show create form page.
     *
     * @param Request $request
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     * @throws PermissionDenied
     * @throws UnauthorizedException
     */
    public function handleCreate(Request $request)
    {
        $this->init();
        $this->bound();

        if (!$this->crud()->actions()->isAllowed('create')) {
            throw new PermissionDenied();
        }

        if (!$this->can('create')) {
            throw UnauthorizedException::forPermissions(['create']);
        }

        return view($this->viewCrudCreate, [

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

            'crud' => $this->crud(),
            'viewsAbove' => $this->getCreateViewsAbove(),
            'viewsBelow' => $this->getCreateViewsBelow(),
        ]);
    }

    /**
     * Get array of view's objects, that should be rendered above content of `create` view.
     *
     * @return array
     */
    protected function getCreateViewsAbove(): array
    {
        return [];
    }

    /**
     * Get array of view's objects, that should be rendered below content of `create` view.
     *
     * @return array
     */
    protected function getCreateViewsBelow(): array
    {
        return [];
    }

    abstract protected function init();

For interfaces and abstract methods it is generally a good practice to add a @return annotation even if it is just @return void or @return null, so that implementors know what to do in the overridden method.

    abstract protected function bound();

For interfaces and abstract methods it is generally a good practice to add a @return annotation even if it is just @return void or @return null, so that implementors know what to do in the overridden method.

    abstract protected function crud(): CRUD;
    abstract protected function can($action): bool;
}