refactor: move integer id to uuid

composer.json

@@ -41,6 +41,7 @@
         "symfony/string": " 6.1.*",
         "symfony/translation": " 6.1.*",
         "symfony/twig-bundle": " 6.1.*",
+        "symfony/uid": " 6.1.*",
         "symfony/validator": " 6.1.*",
         "symfony/web-link": " 6.1.*",
         "symfony/yaml": " 6.1.*",

config/doctrine/orm/Dinosaur.orm.xml

@@ -10,9 +10,7 @@
         table="dinosaur"
     >
 
-        <id name="id" type="integer" column="id">
-            <generator strategy="AUTO"/>
-        </id>
+        <id name="id" type="uuid" />
 
         <field name="name" column="name" type="string" length="255" nullable="false" unique="false" />
         <field name="gender" column="gender" type="string" length="255" nullable="false" unique="false" />

config/doctrine/orm/Species.orm.xml

@@ -10,9 +10,7 @@
         table="species"
     >
 
-        <id name="id" type="integer" column="id">
-            <generator strategy="AUTO"/>
-        </id>
+        <id name="id" type="uuid" />
 
         <field name="name" column="name" type="string" length="255" nullable="false" unique="true" />
         <field name="habitats" column="habitats" type="simple_array" nullable="false" unique="false" />

migrations/Version20230217084031.php

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DoctrineMigrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * Auto-generated Migration: Please modify to your needs!
+ */
+final class Version20230217084031 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Migrating integer ids to UUIDs';
+    }
+
+    public function up(Schema $schema): void
+    {
+        // this up() migration is auto-generated, please modify it to your needs
+        $this->addSql('ALTER TABLE dinosaur DROP FOREIGN KEY FK_DAEDC56EB2A1D860');
+        $this->addSql('ALTER TABLE dinosaur CHANGE id id BINARY(16) NOT NULL COMMENT \'(DC2Type:uuid)\', CHANGE species_id species_id BINARY(16) DEFAULT NULL COMMENT \'(DC2Type:uuid)\'');
+        $this->addSql('ALTER TABLE species CHANGE id id BINARY(16) NOT NULL COMMENT \'(DC2Type:uuid)\'');
+        $this->addSql('ALTER TABLE dinosaur ADD CONSTRAINT FK_DAEDC56EB2A1D860 FOREIGN KEY (species_id) REFERENCES species (id)');
+    }
+
+    public function down(Schema $schema): void
+    {
+        // this down() migration is auto-generated, please modify it to your needs
+        $this->addSql('ALTER TABLE dinosaur DROP FOREIGN KEY FK_DAEDC56EB2A1D860');
+        $this->addSql('ALTER TABLE dinosaur CHANGE id id INT AUTO_INCREMENT NOT NULL, CHANGE species_id species_id INT DEFAULT NULL');
+        $this->addSql('ALTER TABLE species CHANGE id id INT AUTO_INCREMENT NOT NULL');
+        $this->addSql('ALTER TABLE dinosaur ADD CONSTRAINT FK_DAEDC56EB2A1D860 FOREIGN KEY (species_id) REFERENCES species (id)');
+    }
+}

src/Command/ConsumeCommand.php

@@ -39,7 +39,10 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
             $dinosaur = $dinosaurs[$randomDinoKey];
 
-            $consumeMessage = new Consume($dinosaur->getId(), $dinosaur->getName());
+            $consumeMessage = new Consume(
+                $dinosaur->getId()->toRfc4122(),
+                $dinosaur->getName()
+            );
 
             $this->bus->dispatch($consumeMessage);
         }

src/Controller/DinosaursController.php

@@ -50,7 +50,7 @@ public function list(Request $request, ManagerRegistry $doctrine): Response
     #[Route(
         '/dinosaurs/{id}',
         name: 'app_single_dinosaur',
-        requirements: ['id' => '\d+']
+        requirements: ['id' => '^[0-9a-fA-F]{8}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{12}$']
     )]
     public function single(string $id, ManagerRegistry $doctrine): Response
     {
@@ -71,7 +71,7 @@ public function single(string $id, ManagerRegistry $doctrine): Response
     }
 
     #[Route('/dinosaurs/create', name: 'app_create_dinosaur')]
-    public function create(Request $request, ManagerRegistry $doctrine): Response
+    public function create(Request $request): Response
     {
         $form = $this->createForm(DinosaurType::class);
 
@@ -105,9 +105,9 @@ public function create(Request $request, ManagerRegistry $doctrine): Response
     #[Route(
         '/dinosaurs/{id}/edit',
         name: 'app_edit_dinosaur',
-        requirements: ['id' => '\d+']
+        requirements: ['id' => '^[0-9a-fA-F]{8}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{12}$']
     )]
-    public function edit(Request $request, int $id, ManagerRegistry $doctrine): Response
+    public function edit(Request $request, string $id, ManagerRegistry $doctrine): Response
     {
         $dinosaur = $doctrine
             ->getRepository(Dinosaur::class)
@@ -149,9 +149,9 @@ public function edit(Request $request, int $id, ManagerRegistry $doctrine): Resp
     #[Route(
         '/dinosaurs/{id}/remove',
         name: 'app_remove_dinosaur',
-        requirements: ['id' => '\d+']
+        requirements: ['id' => '^[0-9a-fA-F]{8}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{12}$']
     )]
-    public function remove(int $id): Response
+    public function remove(string $id): Response
     {
         $this->bus->dispatch(new Delete($id));
 

src/Controller/SpeciesController.php

@@ -66,9 +66,9 @@ public function create(Request $request, ManagerRegistry $doctrine): Response
     #[Route(
         '/species/{id}/edit',
         name: 'app_edit_species',
-        requirements: ['id' => '\d+']
+        requirements: ['id' => '^[0-9a-fA-F]{8}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{12}$']
     )]
-    public function edit(Request $request, int $id, ManagerRegistry $doctrine): Response
+    public function edit(Request $request, string $id, ManagerRegistry $doctrine): Response
     {
         $species = $doctrine
             ->getRepository(Species::class)
@@ -108,9 +108,9 @@ public function edit(Request $request, int $id, ManagerRegistry $doctrine): Resp
     #[Route(
         '/species/{id}/remove',
         name: 'app_remove_species',
-        requirements: ['id' => '\d+']
+        requirements: ['id' => '^[0-9a-fA-F]{8}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{12}$']
     )]
-    public function remove(int $id): Response
+    public function remove(string $id): Response
     {
         $this->bus->dispatch(new Delete($id));
 

src/Entity/Dinosaur.php

@@ -2,9 +2,11 @@
 
 namespace App\Entity;
 
+use Symfony\Component\Uid\Uuid;
+
 class Dinosaur
 {
-    private int $id;
+    private ?Uuid $id;
     private string $name;
     private string $gender;
     private Species $species;
@@ -18,8 +20,10 @@ public function __construct(
         Species $species,
         int $age,
         string $eyesColor,
-        Park $park
+        Park $park,
+        ?Uuid $id = null
     ) {
+        $this->id = $id ?? Uuid::v4();
         $this->name = $name;
         $this->gender = $gender;
         $this->species = $species;
@@ -28,11 +32,16 @@ public function __construct(
         $this->park = $park;
     }
 
-    public function getId(): int
+    public function getId(): ?Uuid
     {
         return $this->id;
     }
 
+    public function setId(?Uuid $id): void
+    {
+        $this->id = $id;
+    }
+
     public function getName(): string
     {
         return $this->name;