[Config] Improve the component

ConfigCache.php

@@ -46,7 +46,7 @@ public function __construct($cacheDir, $file, $debug)
      */
     public function __toString()
     {
-        return $this->cacheDir.'/'.$this->file.'.php';
+        return $this->getCacheFile();
     }
 
     /**

FileLocator.php

@@ -36,10 +36,11 @@ public function __construct($paths = array())
     /**
      * Returns a full path for a given file name.
      *
-     * @param mixed  $name        The file name to locate
-     * @param string $currentPath The current path
+     * @param mixed   $name        The file name to locate
+     * @param string  $currentPath The current path
+     * @param Boolean $first       Wether to return the first occurence or an array of filenames
      *
-     * @return string The full path for the file
+     * @return string|array The full path to the file|An array of file paths
      *
      * @throws \InvalidArgumentException When file is not found
      */
@@ -54,21 +55,27 @@ public function locate($name, $currentPath = null, $first = true)
         }
 
         $filepaths = array();
-        if (null !== $currentPath && file_exists($currentPath.DIRECTORY_SEPARATOR.$name)) {
-            $filepaths[] = $currentPath.DIRECTORY_SEPARATOR.$name;
+        if (null !== $currentPath && file_exists($file = $currentPath.DIRECTORY_SEPARATOR.$name)) {
+            if (true === $first) {
+                return $file;
+            }
+            $filepaths[] = $file;
         }
 
         foreach ($this->paths as $path) {
-            if (file_exists($path.DIRECTORY_SEPARATOR.$name)) {
-                $filepaths[] = $path.DIRECTORY_SEPARATOR.$name;
+            if (file_exists($file = $path.DIRECTORY_SEPARATOR.$name)) {
+                if (true === $first) {
+                    return $file;
+                }
+                $filepaths[] = $file;
             }
         }
 
         if (!$filepaths) {
             throw new \InvalidArgumentException(sprintf('The file "%s" does not exist (in: %s%s).', $name, null !== $currentPath ? $currentPath.', ' : '', implode(', ', $this->paths)));
         }
 
-        return true === $first ? $filepaths[0] : $filepaths;
+        return $filepaths;
     }
 
     /**

FileLocatorInterface.php

@@ -19,10 +19,11 @@ interface FileLocatorInterface
     /**
      * Returns a full path for a given file name.
      *
-     * @param mixed  $name        The file name to locate
-     * @param string $currentPath The current path
+     * @param mixed   $name        The file name to locate
+     * @param string  $currentPath The current path
+     * @param Boolean $first       Wether to return the first occurence or an array of filenames
      *
-     * @return string The full path for the file
+     * @return string|array The full path to the file|An array of file paths
      *
      * @throws \InvalidArgumentException When file is not found
      */

Loader/DelegatingLoader.php

@@ -21,11 +21,6 @@
  */
 class DelegatingLoader extends Loader
 {
-    /**
-     * @var LoaderResolverInterface
-     */
-    protected $resolver;
-
     /**
      * Constructor.
      *
@@ -41,6 +36,8 @@ public function __construct(LoaderResolverInterface $resolver)
      *
      * @param mixed  $resource A resource
      * @param string $type     The resource type
+     *
+     * @throws \InvalidArgumentException if no loader is found.
      */
     public function load($resource, $type = null)
     {
@@ -63,12 +60,6 @@ public function load($resource, $type = null)
      */
     public function supports($resource, $type = null)
     {
-        foreach ($this->resolver->getLoaders() as $loader) {
-            if ($loader->supports($resource, $type)) {
-                return true;
-            }
-        }
-
-        return false;
+        return false === $this->resolver->resolve($resource, $type) ? false : true;
     }
 }

Loader/Loader.php

@@ -63,17 +63,18 @@ public function import($resource, $type = null)
      */
     public function resolve($resource, $type = null)
     {
-        $loader = false;
+
         if ($this->supports($resource, $type)) {
-            $loader = $this;
-        } elseif (null !== $this->resolver) {
-            $loader = $this->resolver->resolve($resource, $type);
+            return $this;
         }
 
+        $loader = null === $this->resolver ? false : $this->resolver->resolve($resource, $type);
+
         if (false === $loader) {
             throw new \InvalidArgumentException(sprintf('Unable to load the "%s" resource.', is_string($resource) ? $resource : (is_object($resource) ? get_class($resource) : 'RESOURCE')));
         }
 
         return $loader;
     }
+
 }

Loader/LoaderInterface.php

@@ -49,4 +49,5 @@ function getResolver();
      * @param LoaderResolver $resolver A LoaderResolver instance
      */
     function setResolver(LoaderResolver $resolver);
+
 }

Resource/FileResource.php

@@ -53,7 +53,7 @@ public function getResource()
     /**
      * Returns true if the resource has not been updated since the given timestamp.
      *
-     * @param timestamp $timestamp The last time the resource was loaded
+     * @param integer $timestamp The last time the resource was loaded
      *
      * @return Boolean true if the resource has not been updated, false otherwise
      */

Resource/ResourceInterface.php

@@ -28,7 +28,7 @@ function __toString();
     /**
      * Returns true if the resource has not been updated since the given timestamp.
      *
-     * @param int $timestamp The last time the resource was loaded
+     * @param integer $timestamp The last time the resource was loaded
      *
      * @return Boolean true if the resource has not been updated, false otherwise
      */