Fabiang\AsseticBundle\CacheBuster\LastModifiedStrategy::process()

Complexity

Conditions	3
Paths	3

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	3.0052

Importance

Changes

0

<?php
    public function process(AssetInterface $asset, AssetFactory $factory): ?AssetInterface
    {
        $path = $asset->getTargetPath();
        if (null === $path) {
            return null;
        }

        $ext = pathinfo($path, PATHINFO_EXTENSION);

        $lastModified = $factory->getLastModified($asset);
        if (null !== $lastModified) {
            $path = substr_replace(
                $path,
                "$lastModified.$ext",
                -1 * strlen($ext)

It seems like $ext can also be of type array; however, parameter $string of strlen() does only seem to accept string, maybe add an additional type check?

            );
            $asset->setTargetPath($path);

It seems like $path can also be of type array; however, parameter $targetPath of Assetic\Contracts\Asset\AssetInterface::setTargetPath() does only seem to accept string, maybe add an additional type check?

        }
        return null;
    }