merged branch janmarek/patch-1 (PR symfony#5202)

fabpot · fabpot · commit 5e7723fcbb12 · 2012-08-31T11:52:07.000+02:00
Commits ------- 890aea2 FileLocatorInterface used in typehint instead of FileLocator Discussion ---------- FileLocatorInterface used in typehint instead of FileLocator --------------------------------------------------------------------------- by stof at 2012-08-30T22:09:39Z @fabpot this makes sense (and it is BC) --------------------------------------------------------------------------- by mvrhov at 2012-08-31T08:34:17Z What's wrong with Interface hint? I always hint interface when available as this means that I can inject whatever class implementing that interface.
diff --git a/src/Symfony/Component/DependencyInjection/Loader/FileLoader.php b/src/Symfony/Component/DependencyInjection/Loader/FileLoader.php
@@ -13,7 +13,7 @@
 
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\Config\Loader\FileLoader as BaseFileLoader;
-use Symfony\Component\Config\FileLocator;
+use Symfony\Component\Config\FileLocatorInterface;
 
 /**
  * FileLoader is the abstract class used by all built-in loaders that are file based.
@@ -28,9 +28,9 @@ abstract class FileLoader extends BaseFileLoader
      * Constructor.
      *
      * @param ContainerBuilder $container A ContainerBuilder instance
-     * @param FileLocator      $locator   A FileLocator instance
+     * @param FileLocatorInterface      $locator   A FileLocator instance
      */
-    public function __construct(ContainerBuilder $container, FileLocator $locator)
+    public function __construct(ContainerBuilder $container, FileLocatorInterface $locator)
     {
         $this->container = $container;
 
