Helldar\LaravelLangPublisher\Traits\Processors\Deletable

Complexity

Total Complexity

5

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	run()	0	7	2
A	delete()	0	9	3

<?php

namespace Helldar\LaravelLangPublisher\Traits\Processors;

use Helldar\LaravelLangPublisher\Constants\Status;
use Illuminate\Support\Facades\File as IlluminateFile;

trait Deletable
{
    public function run(): array
    {
        $this->delete()
            ? $this->push('*', Status::DELETED)

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

            : $this->push('*', Status::SKIPPED);

        return $this->result();

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

    }

    protected function delete(): bool
    {
        if ($this->isProtected()) {

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

            return false;
        }

        return $this->wantsJson()

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

            ? IlluminateFile::delete($this->targetPath())

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

            : IlluminateFile::deleteDirectory($this->targetPath());
    }
}