Skip to content

Necessary refactorings for Property hooks #7850

Necessary refactorings for Property hooks

Necessary refactorings for Property hooks #7850

Triggered via pull request November 3, 2024 22:47
Status Failure
Total duration 45s
Artifacts

static-analysis.yml

on: pull_request
Matrix: Static Analysis with PHPStan
Matrix: Static Analysis with Psalm
Fit to window
Zoom out
Zoom in

Annotations

25 errors and 1 warning
MismatchingDocblockReturnType: src/Mapping/ClassMetadata.php#L573
src/Mapping/ClassMetadata.php:573:22: MismatchingDocblockReturnType: Docblock has incorrect return type 'Doctrine\ORM\Mapping\LegacyReflectionFields|array<string, ReflectionProperty>', should be 'array<array-key, mixed>' (see https://psalm.dev/142)
ArgumentTypeCoercion: src/Mapping/LegacyReflectionFields.php#L104
src/Mapping/LegacyReflectionFields.php:104:54: ArgumentTypeCoercion: Argument 1 of Doctrine\ORM\Mapping\LegacyReflectionFields::getAccessibleProperty expects class-string, but parent type null|string provided (see https://psalm.dev/193)
PossiblyNullArgument: src/Mapping/LegacyReflectionFields.php#L104
src/Mapping/LegacyReflectionFields.php:104:54: PossiblyNullArgument: Argument 1 of Doctrine\ORM\Mapping\LegacyReflectionFields::getAccessibleProperty cannot be null, possibly null value provided (see https://psalm.dev/078)
UnusedBaselineEntry: src/Internal/Hydration/ObjectHydrator.php#L1
src/Internal/Hydration/ObjectHydrator.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 5 extra entries. (see https://psalm.dev/316)
UnusedBaselineEntry: src/Mapping/ClassMetadata.php#L1
src/Mapping/ClassMetadata.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullArgument" has 1 extra entry. (see https://psalm.dev/316)
UnusedBaselineEntry: src/Mapping/ClassMetadata.php#L1
src/Mapping/ClassMetadata.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 5 extra entries. (see https://psalm.dev/316)
UnusedBaselineEntry: src/PersistentCollection.php#L1
src/PersistentCollection.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 2 extra entries. (see https://psalm.dev/316)
UnusedBaselineEntry: src/Persisters/Entity/BasicEntityPersister.php#L1
src/Persisters/Entity/BasicEntityPersister.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 7 extra entries. (see https://psalm.dev/316)
UnusedBaselineEntry: src/UnitOfWork.php#L1
src/UnitOfWork.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 21 extra entries. (see https://psalm.dev/316)
Static Analysis with Psalm (3.8.2)
Process completed with exit code 2.
MismatchingDocblockReturnType: src/Mapping/ClassMetadata.php#L573
src/Mapping/ClassMetadata.php:573:22: MismatchingDocblockReturnType: Docblock has incorrect return type 'Doctrine\ORM\Mapping\LegacyReflectionFields|array<string, ReflectionProperty>', should be 'array<array-key, mixed>' (see https://psalm.dev/142)
ArgumentTypeCoercion: src/Mapping/LegacyReflectionFields.php#L104
src/Mapping/LegacyReflectionFields.php:104:54: ArgumentTypeCoercion: Argument 1 of Doctrine\ORM\Mapping\LegacyReflectionFields::getAccessibleProperty expects class-string, but parent type null|string provided (see https://psalm.dev/193)
PossiblyNullArgument: src/Mapping/LegacyReflectionFields.php#L104
src/Mapping/LegacyReflectionFields.php:104:54: PossiblyNullArgument: Argument 1 of Doctrine\ORM\Mapping\LegacyReflectionFields::getAccessibleProperty cannot be null, possibly null value provided (see https://psalm.dev/078)
UnusedBaselineEntry: src/Internal/Hydration/ObjectHydrator.php#L1
src/Internal/Hydration/ObjectHydrator.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 5 extra entries. (see https://psalm.dev/316)
UnusedBaselineEntry: src/Mapping/ClassMetadata.php#L1
src/Mapping/ClassMetadata.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullArgument" has 1 extra entry. (see https://psalm.dev/316)
UnusedBaselineEntry: src/Mapping/ClassMetadata.php#L1
src/Mapping/ClassMetadata.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 5 extra entries. (see https://psalm.dev/316)
UnusedBaselineEntry: src/PersistentCollection.php#L1
src/PersistentCollection.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 2 extra entries. (see https://psalm.dev/316)
UnusedBaselineEntry: src/Persisters/Entity/BasicEntityPersister.php#L1
src/Persisters/Entity/BasicEntityPersister.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 7 extra entries. (see https://psalm.dev/316)
UnusedBaselineEntry: src/UnitOfWork.php#L1
src/UnitOfWork.php:0:0: UnusedBaselineEntry: Baseline for issue "PossiblyNullReference" has 21 extra entries. (see https://psalm.dev/316)
Static Analysis with Psalm (default)
Process completed with exit code 2.
Static Analysis with PHPStan (default, phpstan.neon): src/Mapping/ClassMetadata.php#L575
PHPDoc tag @return with type array<string, ReflectionProperty>|Doctrine\ORM\Mapping\LegacyReflectionFields is not subtype of native type array.
Static Analysis with PHPStan (default, phpstan.neon)
Process completed with exit code 1.
Static Analysis with PHPStan (3.8.2, phpstan-dbal3.neon)
The job was canceled because "default_phpstan_neon" failed.
Static Analysis with PHPStan (3.8.2, phpstan-dbal3.neon): src/Mapping/ClassMetadata.php#L575
PHPDoc tag @return with type array<string, ReflectionProperty>|Doctrine\ORM\Mapping\LegacyReflectionFields is not subtype of native type array.
Static Analysis with PHPStan (3.8.2, phpstan-dbal3.neon)
The operation was canceled.
Static Analysis with PHPStan (default, phpstan.neon)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/