Skip to content

Commit

Permalink
Fix: Frozen bag warning when loading services after container is dump…
Browse files Browse the repository at this point in the history
…ed (#63)
  • Loading branch information
RikudouSage committed Mar 14, 2024
1 parent fc18275 commit 9180122
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion src/DependencyInjection/UnleashClientExtension.php
Expand Up @@ -19,6 +19,8 @@
*/
final class UnleashClientExtension extends Extension
{
private bool $servicesYamlLoaded = false;

/**
* @param array<string,mixed> $configs
*
Expand All @@ -32,6 +34,7 @@ public function load(array $configs, ContainerBuilder $container): void
if (interface_exists(ExtensionInterface::class)) {
$loader->load('twig.yaml');
}
$this->servicesYamlLoaded = true;

$configs = $this->processConfiguration($this->getConfiguration([], $container), $configs);
$container->setParameter('unleash.client.internal.service_configs', [
Expand Down Expand Up @@ -82,7 +85,10 @@ public function load(array $configs, ContainerBuilder $container): void
public function getConfiguration(array $config, ContainerBuilder $container): Configuration
{
$loader = new YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
$loader->load('services.yaml');
if (!$this->servicesYamlLoaded) {
$loader->load('services.yaml');
$this->servicesYamlLoaded = true;
}

$handlerNames = [];
foreach ($this->getDefaultStrategyHandlers($container) as $defaultStrategyHandler) {
Expand Down

0 comments on commit 9180122

Please sign in to comment.