Yaro\Jarboe\Table\Actions\DeleteAction

Complexity

Total Complexity

7

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

26.67%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	render()	0	10	2
A	isAllowed()	0	9	3
A	shouldRender()	0	8	2

<?php

namespace Yaro\Jarboe\Table\Actions;

class DeleteAction extends AbstractAction
{
    protected $ident = 'delete';

    public function render($model = null)
    {
        $crud = $this->crud();
        $isVisible = parent::shouldRender($model);

It seems like you call parent on a different method (shouldRender() instead of render()). Are you sure this is correct? If so, you might want to change this to $this->shouldRender().

        if ($crud->isSoftDeleteEnabled()) {
            $isVisible = !$model->trashed();
        }

        return view('jarboe::crud.actions.delete', compact('crud', 'model', 'isVisible'));
    }

    public function isAllowed($model = null)
    {
        $isAllowed = parent::isAllowed($model);
        if ($this->crud()->isSoftDeleteEnabled()) {
            return $isAllowed && !$model->trashed();
        }

        return $isAllowed;
    }

    public function shouldRender($model = null)
    {
        if ($this->crud()->isSoftDeleteEnabled()) {
            return true;
        }

        return parent::shouldRender($model);
    }
}