Helldar\LaravelLangPublisher\Console\Remove

Complexity

Total Complexity

10

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Test Coverage

Coverage

89.66%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	ran()	0	14	3
A	processor()	0	2	1
A	delete()	0	8	2
A	deleteDirectory()	0	13	2
A	deleteFile()	0	13	2

<?php

namespace Helldar\LaravelLangPublisher\Console;

use Helldar\LaravelLangPublisher\Constants\Status;
use Helldar\LaravelLangPublisher\Contracts\Processor;
use Helldar\LaravelLangPublisher\Support\Actions\Remove as Action;
use Illuminate\Support\Facades\File;

final class Remove extends BaseCommand
{
    protected $signature = 'lang:rm'
    . ' {locales?* : Space-separated list of, eg: de tk it}';

    protected $description = 'Remove localizations.';

    protected $action = Action::class;

    protected function ran(): void
    {
        $this->log('Starting processing of the locales list...');

        foreach ($this->locales() as $locale) {
            $this->log('Localization handling: ' . $locale);

            $this->validateLocale($locale);

            $this->processing($locale, $locale);

            $status = $this->doesntProtect($locale) ? $this->delete($locale) : Status::SKIPPED;

            $this->processed($locale, $locale, $status);
        }
    }

    protected function delete(string $locale): string
    {
        $this->log('Removing json and php localization files:', $locale);

        $status_dir  = $this->deleteDirectory($locale);
        $status_file = $this->deleteFile($locale);

        return $status_dir === $status_file ? $status_dir : Status::DELETED;
    }

    protected function deleteDirectory(string $locale): string
    {
        $this->log('Removing the localization directory for the locale:', $locale);

        $path = $this->pathTarget($locale);

        if (File::exists($path)) {
            File::deleteDirectory($path);

            return Status::DELETED;
        }

        return Status::SKIPPED;
    }

    protected function deleteFile(string $locale): string
    {
        $this->log('Removing the json localization file for the locale:', $locale);

        $path = $this->pathTargetFull($locale, null, true);

        if (File::exists($path)) {
            File::delete($path);

            return Status::DELETED;
        }

        return Status::SKIPPED;
    }

    protected function processor(): Processor
    {
    }

In this branch, the function will implicitly return null which is incompatible with the type-hinted return Helldar\LaravelLangPublisher\Contracts\Processor. Consider adding a return statement or allowing null as return value.

}