Only require Twig when profiler is enabled

Author: ruudk

DependencyInjection/Configuration.php

@@ -7,6 +7,7 @@
 use Http\Message\CookieJar;
 use Http\Message\Formatter;
 use Http\Message\StreamFactory;
+use InvalidArgumentException;
 use Psr\Cache\CacheItemPoolInterface;
 use Psr\Log\LoggerInterface;
 use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
@@ -137,6 +138,16 @@ public function getConfigTreeBuilder()
                         ->booleanNode('enabled')
                             ->info('Turn the toolbar on or off. Defaults to kernel debug mode.')
                             ->defaultValue($this->debug)
+                            ->validate()
+                                ->ifTrue()
+                                ->then(function ($v) {
+                                    if (!class_exists(\Twig_Environment::class)) {
+                                        throw new InvalidArgumentException('Twig needs to be installed when enabling the profiler.');
+                                    }
+
+                                    return $v;
+                                })
+                                ->end()
                         ->end()
                         ->scalarNode('formatter')->defaultNull()->end()
                         ->integerNode('captured_body_length')

composer.json

@@ -31,8 +31,7 @@
         "symfony/dependency-injection": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1",
         "symfony/event-dispatcher": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1",
         "symfony/http-kernel": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1",
-        "symfony/options-resolver": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1",
-        "twig/twig": "^1.36 || ^2.6"
+        "symfony/options-resolver": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1"
     },
     "require-dev": {
         "guzzlehttp/psr7": "^1.0",
@@ -50,13 +49,15 @@
         "symfony/phpunit-bridge": "^3.4 || ^4.2",
         "symfony/stopwatch": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1",
         "symfony/twig-bundle": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1",
-        "symfony/web-profiler-bundle": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1"
+        "symfony/web-profiler-bundle": "^2.8.49 || ^3.0.9 || ^3.1.10 || ^3.2.14 || ^3.3.18 || ^3.4.20 || ^4.0.15 || ^4.1.9 || ^4.2.1",
+        "twig/twig": "^1.36 || ^2.6"
     },
     "conflict": {
         "php-http/guzzle6-adapter": "<1.1"
     },
     "suggest": {
-        "php-http/mock-client": "Add this to your require-dev section to mock HTTP responses easily"
+        "php-http/mock-client": "Add this to your require-dev section to mock HTTP responses easily",
+        "twig/twig": "Add this to your require-dev section if you want to use the Symfony Profiler"
     },
     "autoload": {
         "psr-4": {