From a0732003b90b7d0fe235971be8760de3f6270ff1 Mon Sep 17 00:00:00 2001 From: Tomas Date: Wed, 14 Dec 2022 10:45:41 +0100 Subject: [PATCH] Clean-up code after PR10288 merge --- lib/Doctrine/ORM/Mapping/DiscriminatorColumn.php | 1 + lib/Doctrine/ORM/Mapping/Driver/AttributeDriver.php | 11 ++++++----- psalm-baseline.xml | 2 +- 3 files changed, 8 insertions(+), 6 deletions(-) diff --git a/lib/Doctrine/ORM/Mapping/DiscriminatorColumn.php b/lib/Doctrine/ORM/Mapping/DiscriminatorColumn.php index 2a762461e4e..b3f1a20a915 100644 --- a/lib/Doctrine/ORM/Mapping/DiscriminatorColumn.php +++ b/lib/Doctrine/ORM/Mapping/DiscriminatorColumn.php @@ -14,6 +14,7 @@ public function __construct( public readonly string|null $type = null, public readonly int|null $length = null, public readonly string|null $columnDefinition = null, + /** @var class-string<\BackedEnum>|null */ public readonly string|null $enumType = null, ) { } diff --git a/lib/Doctrine/ORM/Mapping/Driver/AttributeDriver.php b/lib/Doctrine/ORM/Mapping/Driver/AttributeDriver.php index 79d543044c2..1d08477b203 100644 --- a/lib/Doctrine/ORM/Mapping/Driver/AttributeDriver.php +++ b/lib/Doctrine/ORM/Mapping/Driver/AttributeDriver.php @@ -207,14 +207,15 @@ public function loadMetadataForClass(string $className, PersistenceClassMetadata // Evaluate DiscriminatorColumn attribute if (isset($classAttributes[Mapping\DiscriminatorColumn::class])) { $discrColumnAttribute = $classAttributes[Mapping\DiscriminatorColumn::class]; + assert($discrColumnAttribute instanceof Mapping\DiscriminatorColumn); $metadata->setDiscriminatorColumn( [ - 'name' => isset($discrColumnAttribute->name) ? (string) $discrColumnAttribute->name : null, - 'type' => isset($discrColumnAttribute->type) ? (string) $discrColumnAttribute->type : 'string', - 'length' => isset($discrColumnAttribute->length) ? (int) $discrColumnAttribute->length : 255, - 'columnDefinition' => isset($discrColumnAttribute->columnDefinition) ? (string) $discrColumnAttribute->columnDefinition : null, - 'enumType' => isset($discrColumnAttribute->enumType) ? (string) $discrColumnAttribute->enumType : null, + 'name' => $discrColumnAttribute->name, + 'type' => $discrColumnAttribute->type ?? 'string', + 'length' => $discrColumnAttribute->length ?? 255, + 'columnDefinition' => $discrColumnAttribute->columnDefinition, + 'enumType' => $discrColumnAttribute->enumType, ], ); } else { diff --git a/psalm-baseline.xml b/psalm-baseline.xml index c6ab26ad541..eb494cdd163 100644 --- a/psalm-baseline.xml +++ b/psalm-baseline.xml @@ -502,7 +502,7 @@ - + $value[0] $value[0]