Support short UUID in Ramsey bridge

[dkarlovi]

We could add rudimentary support in the Ramsey UUID normalizer for the short UUID variant: https://packagist.org/packages/pascaldevink/shortuuid

[soyuka]

It's really easy to add this in one's own project no? Not sure we need to add this to core. That said, if you read this please add 👍 on the issue if there's enough people we may consider it :).

[ParmentierChristophe]

we can't implement this, because this test not working :

    public function testFailDenormalizeUuid()
    {
        $this->expectException(\ApiPlatform\Core\Exception\InvalidIdentifierException::class);

        $uuid = 'notanuuid';
        $normalizer = new ShortUuidNormalizer();
        $this->assertTrue($normalizer->supportsDenormalization($uuid, ShortUuid::class));
        $normalizer->denormalize($uuid, ShortUuid::class);
   
    }

[soyuka]

see pascaldevink/shortuuid#29

[thib92]

Would love to take that one. Any idea how this should be configured? I'm guessing it should be an opt-in configuration option?

[soyuka]

We can't implement this yet because an invalid uuid doesn't throw on the upstream library. see related links, mb you can try to patch there first

[thib92]

Oh good to know, I'll see what I can do. Thanks @soyuka