Merge pull request 'Improves game importer' (#16) from develop into master

Reviewed-on: #16

src/php/Importer/GameImporter.php

@@ -84,6 +84,9 @@ final class GameImporter
         $totalRows = 0;
         $addedAmount = 0;
         
+        $foundGames = [];
+        $rowValues = [];
+
         foreach ($worksheet->getRowIterator(self::HEADER_ROW_INDEX + 1) as $row) {
             $totalRows++;
             $values = [
@@ -96,6 +99,7 @@ final class GameImporter
 
             foreach ($columnDefinitions as $columnIndex => $attribute) {
                 $value = $worksheet->getCell(sprintf('%s%d', $columnIndex, $row->getRowIndex()))->getValueString();
+                $value = trim($value);
                 
                 switch(KeyAttribute::from($attribute)) {
                     case KeyAttribute::NONE:
@@ -123,11 +127,19 @@ final class GameImporter
                 continue;
             }
             
+            $game = array_key_exists($values['name'], $foundGames) ? $foundGames[$values['name']] : null;
+            
+            if ($game === null) {
                 $game = $this->entityManager->getRepository(Game::class)->findOneBy([ 'name' => $values['name'] ]);
+            }
+            
             if ($game === null) {
                 $game = new Game($values['name']);
+                $this->entityManager->persist($game);
             }
             
+            $foundGames[$values['name']] = $game;
+
             $key = new Key(
                 $game,
                 $list,
@@ -137,7 +149,6 @@ final class GameImporter
                 $values['from'],
             );
 
-            $this->entityManager->persist($game);
             $this->entityManager->persist($key);
             $addedAmount++;
         }