Helldar\LaravelLangPublisher\Support\PathJson

Complexity

Total Complexity

7

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	real()	0	3	1
A	source()	0	7	1
A	getPathForEnglish()	0	5	2
A	clean()	0	5	2
A	target()	0	6	1

<?php

namespace Helldar\LaravelLangPublisher\Support;

use Helldar\LaravelLangPublisher\Contracts\Path as PathContract;

The type Helldar\LaravelLangPublisher\Contracts\Path was not found. Maybe you did not declare it correctly or list all dependencies?

use Helldar\LaravelLangPublisher\Facades\Config as ConfigFacade;

final class PathJson implements PathContract
{
    protected $extension = '.json';

    /**
     * Returns a direct link to the folder with the source localization files.
     *
     * If the file name is specified, a full link to the file will be returned,
     * otherwise a direct link to the folder.
     *
     * @param string|null $locale
     * @param string|null $filename
     *
     * @return string
     */
    public function source(string $locale = null, string $filename = null): string
    {
        $locale = $this->getPathForEnglish($locale);

It seems like $locale can also be of type null; however, parameter $locale of Helldar\LaravelLangPublisher\Support\PathJson::getPathForEnglish() does only seem to accept string, maybe add an additional type check?

        $locale = $this->clean($locale);

        return $this->real(
            ConfigFacade::getVendorPath() . '/../json' . $locale . $this->extension
        );
    }

    /**
     * Returns the direct link to the localization folder or,
     * if the file name is specified, a link to the localization file.
     *
     * If the file name or localization is not specified,
     * the link to the shared folder of all localizations will be returned.
     *
     * @param string|null $locale
     * @param string|null $filename
     *
     * @return string
     */
    public function target(string $locale = null, string $filename = null): string
    {
        dd('qqq');
        $locale = $this->clean($locale);

        return resource_path(self::LANG . $locale . $this->extension);
    }

    protected function real(string $path): string
    {
        return realpath($path);
    }

    protected function clean(string $path = null): ?string
    {
        return $path
            ? self::DIVIDER . ltrim($path, self::DIVIDER)
            : $path;
    }

    protected function getPathForEnglish(string $locale): string
    {
        return $locale === 'en'
            ? '../script/en'
            : $locale;
    }
}