Merge pull request #377 from w-vision/dkarlovi-patch-2

fix: don't double-count the import file
instride-ch · Nov 14, 2022 · c6ea991 · c6ea991
2 parents e3a0f11 + 8cf783b
commit c6ea991
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/DataDefinitionsBundle/Importer/Importer.php b/src/DataDefinitionsBundle/Importer/Importer.php
@@ -107,8 +107,8 @@ public function doImport(ImportDefinitionInterface $definition, $params): array
         }
 
 
-        if ((\is_countable($data) || $data instanceof Countable) && \count($data) > 0) {
-            $this->eventDispatcher->dispatch($definition, 'data_definitions.import.total', \count($data), $params);
+        if ((\is_countable($data) || $data instanceof Countable) && ($count = \count($data)) > 0) {
+            $this->eventDispatcher->dispatch($definition, 'data_definitions.import.total', $count, $params);
         }
 
         if ($runner instanceof ImportStartFinishRunnerInterface) {