Helldar\LaravelLangPublisher\Support\Path\Php

Complexity

Total Complexity

2

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	target()	0	6	1
A	source()	0	8	1

<?php

namespace Helldar\LaravelLangPublisher\Support\Path;

use Helldar\LaravelLangPublisher\Facades\Config;

final class Php extends BasePath
{
    /**
     * 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\Path\BasePath::getPathForEnglish() does only seem to accept string, maybe add an additional type check?

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

        return $this->real(
            Config::getVendorPath() . $locale . $filename
        );
    }

    /**
     * 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
    {
        $locale   = $this->clean($locale);
        $filename = $this->clean($filename);

        return resource_path(self::LANG . $locale . $filename);
    }
}