Merge pull request #128 from Jeroen-G/fix-binding

Fix elastic client builder binding
Jeroen-G · Oct 31, 2022 · dbe2531 · dbe2531
2 parents 2945fdc + 44ea512
commit dbe2531
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/changelog.md b/changelog.md
@@ -6,6 +6,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+## [3.3.1]
+
+### Fixes
+- Bug returned class in Elastic client binding in the service provider.
+
 ## [3.3]
 
 ### Added

diff --git a/src/ExplorerServiceProvider.php b/src/ExplorerServiceProvider.php
@@ -34,7 +34,7 @@ public function boot(): void
         }
 
         $this->app->bind(ElasticClientFactory::class, function () {
-            return ElasticClientBuilder::fromConfig(config())->build();
+            return new ElasticClientFactory(ElasticClientBuilder::fromConfig(config())->build());
         });
 
         $this->app->bind(IndexAdapterInterface::class, ElasticIndexAdapter::class);

diff --git a/src/Infrastructure/Elastic/ElasticClientBuilder.php b/src/Infrastructure/Elastic/ElasticClientBuilder.php
@@ -16,7 +16,6 @@ public static function fromConfig(Repository $config): ClientBuilder
     {
         $builder = ClientBuilder::create();
 
-
         $hostConnectionProperties = array_filter(
             $config->get('explorer.connection'),
             static fn ($key) => in_array($key, self::HOST_KEYS, true),