Core: Support short UUID in Ramsey bridge

Created on 6 Sep 2018 · 6Comments · Source: api-platform/core

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

Source

dkarlovi

Most helpful comment

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 :+1: on the issue if there's enough people we may consider it :).

soyuka on 7 Sep 2018

👍4

All 6 comments

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 :+1: on the issue if there's enough people we may consider it :).

soyuka on 7 Sep 2018

👍4

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);

    }