andrey-helldar / laravel-lang-publisher

src/Traits/Processors/Deletable.php

<?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());
    }
}