Thinktomorrow\Chief\Management\Application\DeleteManagedModel

Complexity

Total Complexity

2

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	13	2

<?php

namespace Thinktomorrow\Chief\Management\Application;

use Thinktomorrow\Chief\Audit\Audit;
use Thinktomorrow\Chief\States\PageState;
use Thinktomorrow\Chief\Management\ManagedModel;
use Thinktomorrow\Chief\States\State\StatefulContract;

class DeleteManagedModel
{
    public function handle(ManagedModel $model)
    {
        // For stateful transitions we will apply this deletion as a state
        if($model instanceof StatefulContract) {
            (new PageState($model))->apply('delete');
            $model->save();

The method save() does not exist on Thinktomorrow\Chief\States\State\StatefulContract. It seems like you code against a sub-type of said class. However, the method does not exist in Thinktomorrow\Chief\Tests\Feature\Common\DummyStatefulContract. Are you sure you never get one of those?

The method save() does not exist on Thinktomorrow\Chief\Management\ManagedModel. Since it exists in all sub-types, consider adding an abstract or default implementation to Thinktomorrow\Chief\Management\ManagedModel.

        }

        Audit::activity()
            ->performedOn($model)
            ->log('deleted');

        $model->delete();

The method delete() does not exist on Thinktomorrow\Chief\Management\ManagedModel. Since it exists in all sub-types, consider adding an abstract or default implementation to Thinktomorrow\Chief\Management\ManagedModel.

    }
}