Allow UUIDs for Blameable fields

[fwolfsjaeger]

Feature request: #2641

I'm using UUID as primary key fields and I would like to store the users' UUID in the createdBy/updatedBy fields using the Blameable annotation/attribute.

To allow UUIDs, I've extended the validTypes array in the annotation/XML/YAML driver with 'uuid'.

Moved to separate PR: #2645
In order to convert the given value to whatever type is required, in my case UuidV6 from the Symfony UID component, I've amended the functionality of src/AbstractTrackingListener.php. I'm checking for a setter method to use that one instead of setting the property value directly.

[fwolfsjaeger]

I think it would make sense to move the src/AbstractTrackingListener.php changes to another PR and make it configurable. Any objections?

[fwolfsjaeger]

I've added the CHANGELOG.md entry, but without requiring ramsey/uuid-doctrine this can't be tested. I can add it as a DEV requirement if you want?

[fwolfsjaeger]

Do you require/want any additional changes?

[phansys]

Do you require/want any additional changes?

Yes. Please, review the failing CI jobs. The pipeline must succeed in order to merge.

[fwolfsjaeger]

It seems that ramsey/uuid-doctrine requires PHP 7.4 or higher.

[fwolfsjaeger]

I've updated the branch and moved the CHANGELOG.md entry to [Unreleased].

[mbabker]

It seems that ramsey/uuid-doctrine requires PHP 7.4 or higher.

Instead of using that package, the tests I set up looking at #2701 uses Symfony's UID and Doctrine Bridge packages (the latter of which is already pulled in thanks to the dev dependency chain).

The PHP version constraint isn't an issue anymore, but, I'd say we'd want to support both uuid and ulid types in all extensions (both ramsey/uuid-doctrine and Symfony use uuid as the name for that type, so implicitly supporting one should explicitly support both), and using the Symfony packages would let us test support for both types.

[fwolfsjaeger]

When I created this PR, Doctrine was mostly using ramsey/uuid-doctrine for UUIDs. As of now all my projects are using symfony/uid as well.

I've added ulid as allowed type and replaced the DEV package now.

The packages are interchangeable, you just have to replace the generator in the annotation/attribute as well.

[fwolfsjaeger]

@mbabker Would you mind taking a look at #2645 as well?

[franmomu]

Thanks @fwolfsjaeger! and @mbabker