Replies: 4 comments · 11 replies
-
|
I don’t see how it could happen 🤔 could you provide a reproducer? BTW UUID can be generated client-side, which avoids declaring your identifier nullable, which is better. |
Beta Was this translation helpful? Give feedback.
-
|
It has something to do with this UuidFactory component: namespace Symfony\Component\Uid\Factory;
use Symfony\Component\Uid\Uuid;
use Symfony\Component\Uid\UuidV1;
use Symfony\Component\Uid\UuidV4;
use Symfony\Component\Uid\UuidV5;
use Symfony\Component\Uid\UuidV6;
class UuidFactory
{
private string $defaultClass;
private string $timeBasedClass;
private string $nameBasedClass;
private string $randomBasedClass;
private ?Uuid $timeBasedNode;
private ?Uuid $nameBasedNamespace;
public function __construct(string|int $defaultClass = UuidV6::class, string|int $timeBasedClass = UuidV6::class, string|int $nameBasedClass = UuidV5::class, string|int $randomBasedClass = UuidV4::class, Uuid|string $timeBasedNode = null, Uuid|string $nameBasedNamespace = null)
{
if (null !== $timeBasedNode && !$timeBasedNode instanceof Uuid) {
$timeBasedNode = Uuid::fromString($timeBasedNode);
}
if (null !== $nameBasedNamespace) {
$nameBasedNamespace = $this->getNamespace($nameBasedNamespace);
}
$this->defaultClass = is_numeric($defaultClass) ? Uuid::class.'V'.$defaultClass : $defaultClass;
$this->timeBasedClass = is_numeric($timeBasedClass) ? Uuid::class.'V'.$timeBasedClass : $timeBasedClass;
$this->nameBasedClass = is_numeric($nameBasedClass) ? Uuid::class.'V'.$nameBasedClass : $nameBasedClass;
$this->randomBasedClass = is_numeric($randomBasedClass) ? Uuid::class.'V'.$randomBasedClass : $randomBasedClass;
$this->timeBasedNode = $timeBasedNode;
$this->nameBasedNamespace = $nameBasedNamespace;
}
public function create(): Uuid
{
$class = $this->defaultClass;
return new $class();
}The UuidGenerator class is instantiated with namespace Symfony\Bridge\Doctrine\IdGenerator;
use Doctrine\ORM\EntityManager;
use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\Id\AbstractIdGenerator;
use Symfony\Component\Uid\Factory\NameBasedUuidFactory;
use Symfony\Component\Uid\Factory\RandomBasedUuidFactory;
use Symfony\Component\Uid\Factory\TimeBasedUuidFactory;
use Symfony\Component\Uid\Factory\UuidFactory;
use Symfony\Component\Uid\Uuid;
final class UuidGenerator extends AbstractIdGenerator
{
private readonly UuidFactory $protoFactory;
private UuidFactory|NameBasedUuidFactory|RandomBasedUuidFactory|TimeBasedUuidFactory $factory;
private ?string $entityGetter = null;
public function __construct(UuidFactory $factory = null)
{
$this->protoFactory = $this->factory = $factory ?? new UuidFactory();
}When I dump the $factory parameter it is NULL: public function __construct(UuidFactory $factory = null)
{
dump($factory);
$this->protoFactory = $this->factory = $factory ?? new UuidFactory();
} |
Beta Was this translation helpful? Give feedback.
-
|
The work-around is to not use the auto-generation in the entity: #[ORM\Id]
#[ORM\Column(type: UuidType::NAME, unique: true)]
private ?Uuid $id = null;
public function __construct()
{
$this->id = Uuid::v7();
}It works fine with no problems. |
Beta Was this translation helpful? Give feedback.
-
|
OK, but that has no bearing on the issue. |
Beta Was this translation helpful? Give feedback.
-
|
I think this workaround is no longer needed Symfony fixed it and now generates Uuids v7 |
Beta Was this translation helpful? Give feedback.
-
|
Nothing is fixed for me, still generating v6 by default. |
Beta Was this translation helpful? Give feedback.
-
|
I tried to created a own declare(strict_types=1);
namespace App\Entity\Doctrine;
use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\Id\AbstractIdGenerator;
use Symfony\Component\Uid\Uuid;
use Symfony\Component\Uid\UuidV7;
class UuidV7Generator extends AbstractIdGenerator
{
public function generateId(EntityManagerInterface $em, $entity): UuidV7
{
return Uuid::v7();
}
} #[ORM\Id]
#[ORM\Column(type: UuidType::NAME, unique: true)]
#[ORM\GeneratedValue(strategy: 'CUSTOM')]
#[ORM\CustomIdGenerator(class: UuidV7Generator::class)]
public readonly Uuid $id;With this Workaround a UuidV7 was saved in the Database. |
Beta Was this translation helpful? Give feedback.
-
|
Hi everyone, Same problem on our project, even with redeclaring manually uuid.factory:
class: 'Symfony\Component\Uid\Factory\UuidFactory'
arguments:
$defaultClass: 7
$timeBasedClass: 7
doctrine.uuid_generator:
class: 'Symfony\Bridge\Doctrine\IdGenerator\UuidGenerator'
arguments:
$factory: '@uuid.factory' |
Beta Was this translation helpful? Give feedback.
-
|
Seeing that this impacts many people I spent some time investigating what’s going on. First, using the DoctrineBridge
If you don’t care about those issues, you need to register So, for the final answer: services:
Symfony\Bridge\Doctrine\IdGenerator\UuidGenerator: ~ |
Beta Was this translation helpful? Give feedback.
-
|
Many thanks for the explanations 🙂 |
Beta Was this translation helpful? Give feedback.
-
|
@MatTheCat Thanks man! This is the correct answer for me! |
Beta Was this translation helpful? Give feedback.
-
|
You can try to use the next one: it's work for me |
Beta Was this translation helpful? Give feedback.
-
|
and you get a uuid v7 number? which symfony version? |
Beta Was this translation helpful? Give feedback.
-
|
Can confirm with Symfony 7.3, this also seems to be the code that the maker bundle is generating when using the |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
My UUID config is as follows:
./bin/console debug:config framework uidoutputs:My entity has:
The auto-generated entity ID UUIDs are V6, not V7.
If I inspect any of the UUID primary keys in the database with:
./bin/console uuid:inspect 1eea7e0f-d006-64a2-9024-6b5706aaf847It outputs:
Question
How does one configure the Doctrine Bridge UUID generator to generate V7 UUIDs?
Beta Was this translation helpful? Give feedback.
All reactions