Merge pull request #1232 from chalasr/dead-layer

Remove unneeded BC layer

Repository/ContainerRepositoryFactory.php

@@ -7,7 +7,6 @@
 use Doctrine\ORM\Mapping\ClassMetadata;
 use Doctrine\ORM\Repository\RepositoryFactory;
 use Doctrine\Persistence\ObjectRepository;
-use InvalidArgumentException;
 use Psr\Container\ContainerInterface;
 use RuntimeException;
 
@@ -25,14 +24,8 @@ final class ContainerRepositoryFactory implements RepositoryFactory
     /**
      * @param ContainerInterface $container A service locator containing the repositories
      */
-    public function __construct(ContainerInterface $container = null)
+    public function __construct(ContainerInterface $container)
     {
-        // When DoctrineBundle requires Symfony 3.3+, this can be removed
-        // and the $container argument can become required.
-        if ($container === null) {
-            throw new InvalidArgumentException(sprintf('The first argument of %s::__construct() is required on Symfony 3.3 or higher.', self::class));
-        }
-
         $this->container = $container;
     }