Merge pull request #919 from xabbuh/issue-908

provide better error when doctrine/orm is missing
doctrine · Feb 6, 2019 · a7fa6ef · a7fa6ef
2 parents e54bf58 + e10c26a
commit a7fa6ef
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -2,11 +2,13 @@
 
 namespace Doctrine\Bundle\DoctrineBundle\DependencyInjection;
 
+use Doctrine\ORM\EntityManager;
 use ReflectionClass;
 use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Symfony\Component\Config\Definition\Builder\NodeDefinition;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
+use Symfony\Component\DependencyInjection\Exception\LogicException;
 
 /**
  * This class contains the configuration information for the bundle
@@ -307,6 +309,10 @@ private function addOrmSection(ArrayNodeDefinition $node)
                 ->arrayNode('orm')
                     ->beforeNormalization()
                         ->ifTrue(static function ($v) {
+                            if (! empty($v) && ! class_exists(EntityManager::class)) {
+                                throw new LogicException('The doctrine/orm package is required when the doctrine.orm config is set.');
+                            }
+
                             return $v === null || (is_array($v) && ! array_key_exists('entity_managers', $v) && ! array_key_exists('entity_manager', $v));
                         })
                         ->then(static function ($v) {