Helldar\LaravelLangPublisher\Packages\Package

Complexity

Total Complexity

8

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	basePath()	0	3	1
A	target()	0	3	1
A	targetPath()	0	11	1
A	targetFilename()	0	7	2
A	has()	0	3	1
A	sourcePath()	0	5	1
A	fileBasename()	0	3	1

<?php

namespace Helldar\LaravelLangPublisher\Packages;

use Helldar\LaravelLangPublisher\Concerns\Contains;
use Helldar\LaravelLangPublisher\Concerns\Logger;
use Helldar\LaravelLangPublisher\Contracts\Package as Contract;
use Helldar\LaravelLangPublisher\Facades\Config;
use Helldar\LaravelLangPublisher\Facades\Path;
use Helldar\Support\Concerns\Makeable;
use Helldar\Support\Facades\Helpers\Filesystem\Directory;

abstract class Package implements Contract
{
    use Contains;
    use Logger;

The trait Helldar\LaravelLangPublisher\Concerns\Logger requires the property $output which is not provided by Helldar\LaravelLangPublisher\Packages\Package.

    use Makeable;

    public function target(): string
    {
        return '/';
    }

    public function has(string $package = null, string $locale = null): bool
    {
        return Directory::exists($this->basePath() . '/' . $this->vendor());
    }

    public function sourcePath(string $package, string $locale): string
    {
        $path = Path::source($package, $locale);

        return $path . '/' . $this->source();

Are you sure $this->source() of type array can be used in concatenation?

    }

    public function targetPath(string $locale, string $filename): string
    {
        $target = Path::clean(Path::target($locale));

        $path = $this->target();

        $name = $this->targetFilename($locale, $filename);

        $filename = Path::clean($path . '/' . $name, true);

        return $target . '/' . $filename;
    }

    public function targetFilename(string $locale, string $filename): string
    {
        if ($this->isJson($filename)) {
            return $locale . '.json';
        }

        return $this->fileBasename($filename);
    }

    protected function basePath(): string
    {
        return Config::basePath();
    }

    protected function fileBasename(string $filename): string
    {
        return Path::basename($filename);
    }
}