regularjack / frontend-bundle

Manifest/Loader/AbstractManifestLoader.php

<?php

namespace Rj\FrontendBundle\Manifest\Loader;

use Rj\FrontendBundle\Manifest\Manifest;

abstract class AbstractManifestLoader implements ManifestLoaderInterface
{
    private $path;
    private $rootKey;

    abstract protected function parse($path);

    public function __construct($path, $rootKey = null)
    {
        if (!is_file($path)) {
            throw new \InvalidArgumentException("The manifest file '$path' could not be found");

As per coding-style, please use concatenation or sprintf for the variable $path instead of interpolation.

        }

        $this->path = $path;
        $this->rootKey = $rootKey;
    }

    /**
     * {@inheritdoc}
     */
    public function load()
    {
        $entries = $this->parse($this->path);

        if (!empty($this->rootKey)) {
            if (!isset($entries[$this->rootKey])) {
                throw new \InvalidArgumentException('Manifest file contains no '.$this->rootKey.' key');
            }

            $entries = $entries[$this->rootKey];
        }

        return new Manifest($entries);
    }

    /**
     * {@inheritdoc}
     */
    public function getPath()
    {
        return $this->path;
    }
}