Enjoys\AssetsCollector\CollectStrategy\Strategy\ManyFilesStrategy::getResult()

Complexity

Conditions	6
Paths	11

Size

Total Lines	42
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	21
CRAP Score	6.0702

Importance

Changes	3
Bugs	1	Features	0

<?php
    public function getResult(): array
    {
        $result = [];

        foreach ($this->assetsCollection as $asset) {
            if ($asset->getPath() === false) {
                continue;
            }

            if ($asset->isUrl()) {
                $result[] = $asset->getPath();
                continue;
            }

            $link = str_replace(
                [
                    $this->environment->getCompileDir(),
                    $this->environment->getProjectDir()
                ]
                ,
                '',
                $asset->getPath()
            );

            $symlink = $this->environment->getCompileDir() . $link;

            try {
                if (!file_exists($symlink)) {
                    $path = pathinfo($symlink, PATHINFO_DIRNAME);
                    Helpers::createDirectory($path);

It seems like $path can also be of type array; however, parameter $path of Enjoys\AssetsCollector\Helpers::createDirectory() does only seem to accept string, maybe add an additional type check?

                    $this->logger->info(sprintf('Create directory %s', $path));

                    symlink($asset->getPath(), $symlink);
                    $this->logger->info(sprintf('Create symlink: %s', $symlink));
                }
            } catch (\Exception  $e) {
                $this->logger->error($e->getMessage());
            }

            $result[] = $this->environment->getBaseUrl() . str_replace(DIRECTORY_SEPARATOR, '/', $link);
        }
        return $result;
    }