Renamed classes.

src/Packagist/WebBundle/Repository/Provider/GitRepositoryProvider.php

@@ -4,11 +4,13 @@ namespace Packagist\WebBundle\Repository\Provider;
 
 use Packagist\WebBundle\Repository\Repository\GitRepository;
 
-class GitRepositoryProvider implements RepositoryProviderInterface
+class GitProvider implements ProviderInterface
 {
     public function getRepository($url)
     {
-        return new GitRepository($url);
+        if($this->supports($url)){
+            return new GitRepository($url);
+        }
     }
 
     public function supports($url)

src/Packagist/WebBundle/Repository/Provider/RepositoryProviderInterface.php → src/Packagist/WebBundle/Repository/Provider/ProviderInterface.php

@@ -2,18 +2,19 @@
 
 namespace Packagist\WebBundle\Repository\Provider;
 
-interface RepositoryProviderInterface
+interface ProviderInterface
 {
     /**
-     * Does the provider support the URL?
+     * Returns whether the provider supports the URL
      * @param string $url
      */
     public function supports($url);
 
     /**
-     * Get the repository for the URL
-     * @param string $url
+     * Get the repository for the URL.
+     * This method is expected to return null if the URL is not supported.
      *
+     * @param string $url
      */
     public function getRepository($url);
 }

src/Packagist/WebBundle/Repository/RepositoryProviderProvider.php → src/Packagist/WebBundle/Repository/RepositoryProvider.php

@@ -4,7 +4,7 @@ namespace Packagist\WebBundle\Repository;
 
 use Packagist\WebBundle\Repository\Provider\RepositoryProviderInterface;
 
-class RepositoryProviderProvider
+class RepositoryProvider implements RepositoryProviderInterface
 {
     protected $providers = array();
 

src/Packagist/WebBundle/Repository/RepositoryProviderInteface.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace Packagist\WebBundle\Repository;
+
+interface RepositoryProviderInterface
+{
+    public function addProvider(RepositoryProviderInterface $provider);
+
+    public function getRepository($url);
+}

src/Packagist/WebBundle/Resources/config/services.yml

@@ -1,8 +1,8 @@
 services:
     git_repository_provider:
-        class: Packagist\WebBundle\Repository\Provider\GitRepositoryProvider
+        class: Packagist\WebBundle\Repository\Provider\GitProvider
         
     repository_provider:
-        class: Packagist\WebBundle\Repository\RepositoryProviderProvider
+        class: Packagist\WebBundle\Repository\RepositoryProvider
         calls:
             - [ addProvider, [ @git_repository_provider ] ]