Compare commits
No commits in common. "ff05142acc7fe2ed6cbef3ddd12458ba95e4f1cf" and "56fee01322e651832e31e3ba042d20c549612835" have entirely different histories.
ff05142acc
...
56fee01322
2 changed files with 7 additions and 47 deletions
|
@ -15,8 +15,6 @@ class Game
|
|||
private int|null $id;
|
||||
#[ORM\Column]
|
||||
private string $name;
|
||||
#[ORM\Column(nullable: true)]
|
||||
private string|null $internalName;
|
||||
|
||||
public function getName(): string
|
||||
{
|
||||
|
@ -27,23 +25,12 @@ class Game
|
|||
{
|
||||
return $this->id;
|
||||
}
|
||||
|
||||
public function getInternalName(): string
|
||||
{
|
||||
return $this->internalName ?? $this->name;
|
||||
}
|
||||
|
||||
public function setInternalName(string $internalName): void
|
||||
{
|
||||
$this->internalName = $internalName;
|
||||
}
|
||||
|
||||
/**
|
||||
* @param string $name
|
||||
*/
|
||||
public function __construct(string $internalName, string $name)
|
||||
public function __construct(string $name)
|
||||
{
|
||||
$this->internalName = $internalName;
|
||||
$this->name = $name;
|
||||
}
|
||||
}
|
|
@ -3,17 +3,14 @@ declare(strict_types=1);
|
|||
|
||||
namespace GamesShop\Importer;
|
||||
|
||||
use Doctrine\Common\Collections\Criteria;
|
||||
use Doctrine\ORM\EntityManagerInterface;
|
||||
use Exception;
|
||||
use GamesShop\ContainerHandler;
|
||||
use GamesShop\Entities\Account\User;
|
||||
use GamesShop\Entities\Games\Game;
|
||||
use GamesShop\Entities\Games\Key;
|
||||
use GamesShop\Entities\Games\KeyAttribute;
|
||||
use GamesShop\Entities\Games\Store;
|
||||
use GamesShop\Entities\GamesList;
|
||||
use Monolog\Logger;
|
||||
use PhpOffice\PhpSpreadsheet\IOFactory;
|
||||
|
||||
final class GameImporter
|
||||
|
@ -130,27 +127,19 @@ final class GameImporter
|
|||
continue;
|
||||
}
|
||||
|
||||
$internalName = $this->prepareGameName($values['name']);
|
||||
$game = array_key_exists($internalName, $foundGames) ? $foundGames[$internalName] : null;
|
||||
|
||||
ContainerHandler::get(Logger::class)->debug('importing game ' . $internalName);
|
||||
$game = array_key_exists($values['name'], $foundGames) ? $foundGames[$values['name']] : null;
|
||||
|
||||
if ($game === null) {
|
||||
$game = $this->entityManager->getRepository(Game::class)->matching(
|
||||
Criteria::create()
|
||||
->where(Criteria::expr()->eq('name', $values['name']))
|
||||
->orWhere(Criteria::expr()->eq('internalName', $internalName))
|
||||
)->first();
|
||||
$game = $this->entityManager->getRepository(Game::class)->findOneBy([ 'name' => $values['name'] ]);
|
||||
}
|
||||
|
||||
if ($game === false) {
|
||||
$game = new Game($internalName, $values['name']);
|
||||
if ($game === null) {
|
||||
$game = new Game($values['name']);
|
||||
$this->entityManager->persist($game);
|
||||
}
|
||||
|
||||
$foundGames[$internalName] = $game;
|
||||
$game->setInternalName($internalName);
|
||||
|
||||
$foundGames[$values['name']] = $game;
|
||||
|
||||
$key = new Key(
|
||||
$game,
|
||||
$list,
|
||||
|
@ -185,20 +174,4 @@ final class GameImporter
|
|||
|
||||
return Store::EXTERNAL;
|
||||
}
|
||||
|
||||
private function prepareGameName(string $name): string
|
||||
{
|
||||
$replacements = [
|
||||
':' => '',
|
||||
'-' => '',
|
||||
',' => '',
|
||||
'!' => '',
|
||||
'?' => '',
|
||||
' ' => ' ',
|
||||
' ' => '_',
|
||||
];
|
||||
|
||||
$name = strtoupper($name);
|
||||
return str_replace(array_keys($replacements), array_values($replacements), $name);
|
||||
}
|
||||
}
|
Loading…
Reference in a new issue