WebComplete\mvc\assets\AssetManager

Complexity

Total Complexity

22

Size/Duplication

Total Lines	157
Duplicated Lines	14.01 %

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	applyJs()	10	10	3
A	applyCss()	10	10	3
A	registerAsset()	0	6	2
A	getLink()	0	10	3
A	__construct()	0	5	1
A	getLinks()	0	8	2
A	publishAsset()	0	14	4
A	getWebDir()	0	5	2
A	getPath()	0	7	2

<?php

namespace WebComplete\mvc\assets;

use Symfony\Component\Filesystem\Filesystem;
use WebComplete\mvc\exception\Exception;

class AssetManager
{

    /**
     * @var AbstractAsset[]
     */
    protected $assets = [];
    /**
     * @var string
     */
    protected $webRoot;
    /**
     * @var string
     */
    protected $assetDirName;
    /**
     * @var Filesystem
     */
    protected $filesystem;

    /**
     * @param Filesystem $filesystem
     * @param string $webRoot
     * @param string $assetDirName
     */
    public function __construct(Filesystem $filesystem, string $webRoot, string $assetDirName)
    {
        $this->filesystem = $filesystem;
        $this->webRoot = \rtrim($webRoot, '/');
        $this->assetDirName = $assetDirName;
    }

    /**
     * @param AbstractAsset $asset
     *
     * @throws \Symfony\Component\Filesystem\Exception\IOException
     */
    public function registerAsset(AbstractAsset $asset)
    {
        if ($asset->publish) {
            $this->publishAsset($asset);
        }
        $this->assets[\get_class($asset)] = $asset;
    }

    /**
     * @param string $assetClass
     * @param $file
     *
     * @return string
     * @throws Exception
     */
    public function getPath(string $assetClass, $file): string
    {
        if (!isset($this->assets[$assetClass])) {
            throw new Exception('Asset ' . $assetClass . ' is not registered');
        }

        return $this->getWebDir($this->assets[$assetClass]) . \ltrim($file, '/');
    }

    /**
     * @return string
     */
    public function applyCss(): string

This method seems to be duplicated in your project.

    {
        $result = [];
        foreach ($this->assets as $asset) {
            $links = $this->getLinks($asset, $asset->css());
            foreach ($links as $link) {
                $result[] = '<link rel="stylesheet" href="' . $link . '">';
            }
        }
        return \implode('', $result);
    }

    /**
     * @return string
     */
    public function applyJs(): string

This method seems to be duplicated in your project.

    {
        $result = [];
        foreach ($this->assets as $asset) {
            $links = $this->getLinks($asset, $asset->js());
            foreach ($links as $link) {
                $result[] = '<script src="' . $link . '"></script>';
            }
        }
        return \implode('', $result);
    }

    /**
     * @param AbstractAsset $asset
     *
     * @throws \Symfony\Component\Filesystem\Exception\IOException
     */
    protected function publishAsset(AbstractAsset $asset)
    {
        $hash = $asset->getHash();
        $dir = $this->webRoot . '/' . $this->assetDirName . '/' . $hash;
        if ($asset->useLinks) {
            if (!$this->filesystem->exists($dir)) {
                $this->filesystem->symlink($asset->getBasePath(), $dir, true);
            }
        } else {
            if ($this->filesystem->exists($dir)) {
                $this->filesystem->remove($dir);
            }
            $this->filesystem->mirror($asset->getBasePath(), $dir);
            $this->filesystem->chmod($dir, 0755, 000, true);
        }
    }

    /**
     * @param AbstractAsset $asset
     *
     * @return string
     */
    private function getWebDir(AbstractAsset $asset): string
    {
        return $asset->publish
            ? '/' . $this->assetDirName . '/' . $asset->getHash() . '/'
            : '/';
    }

    /**
     * @param AbstractAsset $asset
     * @param array $files
     *
     * @return array
     */
    private function getLinks(AbstractAsset $asset, array $files): array
    {
        $result = [];
        $dir = $this->getWebDir($asset);
        foreach ($files as $file) {
            $result[] = $this->getLink($file, $dir);
        }
        return $result;
    }

    /**
     * @param $file
     * @param $dir
     *
     * @return string
     */
    private function getLink($file, $dir): string
    {
        $link = $file;
        if (false === \strpos($file, 'http') && false === \strpos($file, '//')) {
            $filePath = $dir . \ltrim($file, '/');
            $timestamp = \filemtime($this->webRoot . $filePath);
            $link = $filePath . '?' . $timestamp;

Are you sure $timestamp of type integer|false can be used in concatenation?

        }

        return $link;
    }
}