Fix populate command (#10)

* Fixed populate command issue * Fixed populate command issue * Fixed populate command issue
Locastic · Sep 23, 2023 · d0147b4 · d0147b4
1 parent 8eec576
commit d0147b4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Bridge/Elasticsearch/ElasticsearchService.php b/src/Bridge/Elasticsearch/ElasticsearchService.php
@@ -41,7 +41,7 @@ public function bulkCreate(array $items, string $index, array $groups = []): voi
             $params[] = [
                 'index' => [
                     '_index' => $index,
-                    '_id' => $item->getId(),
+                    '_id' => method_exists($item, 'getId')? $item->getId() : null,
                 ],
             ];
             $params[] = $normalizedItem;

diff --git a/src/Resources/config/message_handlers.yaml b/src/Resources/config/message_handlers.yaml
@@ -17,7 +17,7 @@ services:
     Locastic\Loggastic\MessageHandler\PopulateCurrentDataTrackersHandler:
         arguments:
             - '@Doctrine\Persistence\ManagerRegistry'
-            - '@Locastic\Loggastic\Factory\ActivityLogInputFactoryInterface'
+            - '@Locastic\Loggastic\Factory\CurrentDataTrackerInputFactoryInterface'
             - '@serializer.normalizer.object'
             - '@Locastic\Loggastic\Bridge\Elasticsearch\ElasticsearchService'
             - '@Locastic\Loggastic\Bridge\Elasticsearch\Context\ElasticsearchContextFactoryInterface'