Merge pull request 'Improves game importer' (#16) from develop into master
All checks were successful
/ build (push) Successful in 36s
All checks were successful
/ build (push) Successful in 36s
Reviewed-on: #16
This commit is contained in:
commit
ab4ebe93a3
1 changed files with 15 additions and 4 deletions
|
@ -83,6 +83,9 @@ final class GameImporter
|
|||
|
||||
$totalRows = 0;
|
||||
$addedAmount = 0;
|
||||
|
||||
$foundGames = [];
|
||||
$rowValues = [];
|
||||
|
||||
foreach ($worksheet->getRowIterator(self::HEADER_ROW_INDEX + 1) as $row) {
|
||||
$totalRows++;
|
||||
|
@ -96,7 +99,8 @@ 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:
|
||||
break;
|
||||
|
@ -122,11 +126,19 @@ final class GameImporter
|
|||
if (empty($values['key']) || empty($values['name']) || empty($values['store'])) {
|
||||
continue;
|
||||
}
|
||||
|
||||
$game = $this->entityManager->getRepository(Game::class)->findOneBy([ 'name' => $values['name'] ]);
|
||||
|
||||
$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,
|
||||
|
@ -137,7 +149,6 @@ final class GameImporter
|
|||
$values['from'],
|
||||
);
|
||||
|
||||
$this->entityManager->persist($game);
|
||||
$this->entityManager->persist($key);
|
||||
$addedAmount++;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue