Skip to content
This repository was archived by the owner on Aug 22, 2023. It is now read-only.

Commit c16e7b9

Browse files
authored
Merge pull request #89 from ajgarlag/bugfix/chain-activator
Use TraceableChainDecorator in debug mode only
2 parents 16b1061 + bb28aac commit c16e7b9

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

src/DependencyInjection/FlagceptionExtension.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
use Exception;
66
use Flagception\Activator\FeatureActivatorInterface;
7+
use Flagception\Bundle\FlagceptionBundle\Activator\TraceableChainActivator;
78
use Flagception\Bundle\FlagceptionBundle\DependencyInjection\Configurator\ActivatorConfiguratorInterface;
89
use Flagception\Decorator\ContextDecoratorInterface;
910
use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -51,6 +52,11 @@ public function load(array $configs, ContainerBuilder $container)
5152
$configurator->addActivator($container, $config['activators'][$name], $config['features']);
5253
}
5354

55+
if ($container->hasParameter('kernel.debug') && $container->getParameter('kernel.debug')) {
56+
$chainDefinition = $container->getDefinition('flagception.activator.chain_activator');
57+
$chainDefinition->setClass(TraceableChainActivator::class);
58+
}
59+
5460
if (method_exists($container, 'registerForAutoconfiguration') === true) {
5561
$container
5662
->registerForAutoconfiguration(FeatureActivatorInterface::class)

src/Resources/config/services.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ services:
5656
public: false
5757

5858
flagception.activator.chain_activator:
59-
class: Flagception\Bundle\FlagceptionBundle\Activator\TraceableChainActivator
59+
class: Flagception\Activator\ChainActivator
6060
public: false
6161

6262
flagception.decorator.chain_decorator:

0 commit comments

Comments
 (0)