Skip to content

Commit 1991874

Browse files
committed
Refactored IbexaSolrExtension to extend ConfigurableExtension
1 parent 3cb5237 commit 1991874

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

src/bundle/DependencyInjection/IbexaSolrExtension.php

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,12 @@
2020
use Symfony\Component\DependencyInjection\Definition;
2121
use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
2222
use Symfony\Component\DependencyInjection\Reference;
23-
use Symfony\Component\HttpKernel\DependencyInjection\Extension;
23+
use Symfony\Component\HttpKernel\DependencyInjection\ConfigurableExtension;
2424

2525
/**
2626
* @phpstan-type SolrHttpClientConfigArray = array{timeout: int, max_retries: int}
2727
*/
28-
class IbexaSolrExtension extends Extension
28+
class IbexaSolrExtension extends ConfigurableExtension
2929
{
3030
/**
3131
* Main Solr search handler service ID.
@@ -103,15 +103,13 @@ public function getServicePrefix(): string
103103
}
104104

105105
/**
106+
* @param array<string, mixed> $mergedConfig
107+
*
106108
* @throws \InvalidArgumentException When provided tag is not defined in this extension
107109
* @throws \Exception
108110
*/
109-
public function load(array $configs, ContainerBuilder $container): void
111+
public function loadInternal(array $mergedConfig, ContainerBuilder $container): void
110112
{
111-
$configuration = $this->getConfiguration($configs, $container);
112-
assert($configuration !== null);
113-
$config = $this->processConfiguration($configuration, $configs);
114-
115113
// Loading configuration from lib/Resources/config/container
116114
$loader = new YamlFileLoader(
117115
$container,
@@ -125,7 +123,7 @@ public function load(array $configs, ContainerBuilder $container): void
125123
);
126124
$loader->load('services.yml');
127125

128-
$this->processConnectionConfiguration($container, $config);
126+
$this->processConnectionConfiguration($container, $mergedConfig);
129127

130128
$container
131129
->registerForAutoconfiguration(UpdateSerializerInterface::class)

0 commit comments

Comments
 (0)