Skip to content

Commit 67f10a4

Browse files
author
Jeremiah VALERIE
committed
Merge branch '0.11' into 0.12
2 parents bb07b0f + e5d7346 commit 67f10a4

File tree

3 files changed

+29
-18
lines changed

3 files changed

+29
-18
lines changed

src/CacheWarmer/CompileCacheWarmer.php

Lines changed: 19 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,33 +9,43 @@
99

1010
class CompileCacheWarmer implements CacheWarmerInterface
1111
{
12-
/** @var TypeGenerator */
1312
private $typeGenerator;
1413

15-
public function __construct(TypeGenerator $typeGenerator)
14+
private $compiled;
15+
16+
/**
17+
* CompileCacheWarmer constructor.
18+
*
19+
* @param TypeGenerator $typeGenerator
20+
* @param bool $compiled
21+
*/
22+
public function __construct(TypeGenerator $typeGenerator, $compiled = true)
1623
{
1724
$this->typeGenerator = $typeGenerator;
25+
$this->compiled = $compiled;
1826
}
1927

2028
/**
2129
* {@inheritdoc}
2230
*/
2331
public function isOptional()
2432
{
25-
return false;
33+
return !$this->compiled;
2634
}
2735

2836
/**
2937
* {@inheritdoc}
3038
*/
3139
public function warmUp($cacheDir): void
3240
{
33-
// use warm up cache dir if type generator cache dir not already explicitly declare
34-
$baseCacheDir = $this->typeGenerator->getBaseCacheDir();
35-
if (null === $this->typeGenerator->getCacheDir(false)) {
36-
$this->typeGenerator->setBaseCacheDir($cacheDir);
41+
if ($this->compiled) {
42+
// use warm up cache dir if type generator cache dir not already explicitly declare
43+
$baseCacheDir = $this->typeGenerator->getBaseCacheDir();
44+
if (null === $this->typeGenerator->getCacheDir(false)) {
45+
$this->typeGenerator->setBaseCacheDir($cacheDir);
46+
}
47+
$this->typeGenerator->compile(TypeGenerator::MODE_WRITE | TypeGenerator::MODE_OVERRIDE);
48+
$this->typeGenerator->setBaseCacheDir($baseCacheDir);
3749
}
38-
$this->typeGenerator->compile(TypeGenerator::MODE_WRITE | TypeGenerator::MODE_OVERRIDE);
39-
$this->typeGenerator->setBaseCacheDir($baseCacheDir);
4050
}
4151
}

src/DependencyInjection/Configuration.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ private function definitionsSection()
125125
->scalarNode('cache_dir')->defaultNull()->end()
126126
->scalarNode('cache_dir_permissions')->defaultNull()->end()
127127
->booleanNode('use_classloader_listener')->defaultTrue()->end()
128-
->booleanNode('auto_compile')->defaultTrue()->end()
128+
->scalarNode('auto_compile')->defaultTrue()->end()
129129
->booleanNode('show_debug_info')->info('Show some performance stats in extensions')->defaultFalse()->end()
130130
->booleanNode('config_validation')->defaultValue($this->debug)->end()
131131
->append($this->definitionsSchemaSection())

src/DependencyInjection/OverblogGraphQLExtension.php

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -78,14 +78,15 @@ private function loadConfigFiles(ContainerBuilder $container): void
7878

7979
private function setCompilerCacheWarmer(array $config, ContainerBuilder $container): void
8080
{
81-
if ($config['definitions']['auto_compile']) {
82-
$definition = $container->setDefinition(
83-
CompileCacheWarmer::class,
84-
new Definition(CompileCacheWarmer::class)
85-
);
86-
$definition->setArguments([new Reference($this->getAlias().'.cache_compiler')]);
87-
$definition->addTag('kernel.cache_warmer', ['priority' => 50]);
88-
}
81+
$definition = $container->setDefinition(
82+
CompileCacheWarmer::class,
83+
new Definition(CompileCacheWarmer::class)
84+
);
85+
$definition->setArguments([
86+
new Reference($this->getAlias().'.cache_compiler'),
87+
$config['definitions']['auto_compile'],
88+
]);
89+
$definition->addTag('kernel.cache_warmer', ['priority' => 50]);
8990
}
9091

9192
private function setClassLoaderListener(array $config, ContainerBuilder $container): void

0 commit comments

Comments
 (0)