Thinktomorrow\Chief\Management\Application\DeleteManagedModel::handle()

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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.

    }