Skip to content

Necessary refactorings for Property hooks #7847

Necessary refactorings for Property hooks

Necessary refactorings for Property hooks #7847

Triggered via pull request November 3, 2024 20:02
Status Failure
Total duration 46s
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

21 errors and 2 warnings
TooManyTemplateParams: src/Mapping/ClassMetadata.php#L545
src/Mapping/ClassMetadata.php:545:5: TooManyTemplateParams: Doctrine\ORM\Mapping\LegacyReflectionFields<string, ReflectionProperty> has too many template params, expecting 0 (see https://psalm.dev/184)
TooManyTemplateParams: src/Mapping/ClassMetadata.php#L545
src/Mapping/ClassMetadata.php:545:12: TooManyTemplateParams: Doctrine\ORM\Mapping\LegacyReflectionFields<string, ReflectionProperty> has too many template params, expecting 0 (see https://psalm.dev/184)
InvalidReturnType: src/Mapping/ClassMetadata.php#L573
src/Mapping/ClassMetadata.php:573:22: InvalidReturnType: The declared return type 'array<array-key, ReflectionProperty|null>' for Doctrine\ORM\Mapping\ClassMetadata::getReflectionProperties is incorrect, got 'Doctrine\ORM\Mapping\LegacyReflectionFields<string, ReflectionProperty>|array<string, ReflectionProperty>' (see https://psalm.dev/011)
InvalidReturnStatement: src/Mapping/ClassMetadata.php#L577
src/Mapping/ClassMetadata.php:577:16: InvalidReturnStatement: The inferred type 'Doctrine\ORM\Mapping\LegacyReflectionFields<string, ReflectionProperty>|array<string, ReflectionProperty>' does not match the declared return type 'array<array-key, ReflectionProperty|null>' for Doctrine\ORM\Mapping\ClassMetadata::getReflectionProperties (see https://psalm.dev/128)
MissingTemplateParam: src/Mapping/LegacyReflectionFields.php#L21
src/Mapping/LegacyReflectionFields.php:21:41: MissingTemplateParam: Doctrine\ORM\Mapping\LegacyReflectionFields has missing template params when extending ArrayAccess, expecting 2 (see https://psalm.dev/182)
MissingTemplateParam: src/Mapping/LegacyReflectionFields.php#L21
src/Mapping/LegacyReflectionFields.php:21:54: MissingTemplateParam: Doctrine\ORM\Mapping\LegacyReflectionFields has missing template params when extending IteratorAggregate, expecting 2 (see https://psalm.dev/182)
InvalidPropertyAssignmentValue: src/Mapping/LegacyReflectionFields.php#L69
src/Mapping/LegacyReflectionFields.php:69:13: InvalidPropertyAssignmentValue: $this->reflFields with declared type 'array<string, ReflectionProperty>' cannot be assigned type 'non-empty-array<string, ReflectionProperty|null>' (see https://psalm.dev/145)
ArgumentTypeCoercion: src/Mapping/LegacyReflectionFields.php#L69
src/Mapping/LegacyReflectionFields.php:69:70: ArgumentTypeCoercion: Argument 1 of Doctrine\ORM\Mapping\LegacyReflectionFields::getAccessibleProperty expects class-string, but parent type string provided (see https://psalm.dev/193)
PossiblyNullArgument: src/Mapping/LegacyReflectionFields.php#L69
src/Mapping/LegacyReflectionFields.php:69:82: PossiblyNullArgument: Argument 2 of Doctrine\ORM\Mapping\LegacyReflectionFields::getAccessibleProperty cannot be null, possibly null value provided (see https://psalm.dev/078)
InvalidPropertyAssignmentValue: src/Mapping/LegacyReflectionFields.php#L73
src/Mapping/LegacyReflectionFields.php:73:21: InvalidPropertyAssignmentValue: $this->reflFields with declared type 'array<string, ReflectionProperty>' cannot be assigned type 'non-empty-array<string, ReflectionProperty|null>' (see https://psalm.dev/145)
Static Analysis with Psalm (3.8.2)
The job was canceled because "default" failed.
TooManyTemplateParams: src/Mapping/ClassMetadata.php#L545
src/Mapping/ClassMetadata.php:545:5: TooManyTemplateParams: Doctrine\ORM\Mapping\LegacyReflectionFields<string, ReflectionProperty> has too many template params, expecting 0 (see https://psalm.dev/184)
TooManyTemplateParams: src/Mapping/ClassMetadata.php#L545
src/Mapping/ClassMetadata.php:545:12: TooManyTemplateParams: Doctrine\ORM\Mapping\LegacyReflectionFields<string, ReflectionProperty> has too many template params, expecting 0 (see https://psalm.dev/184)
InvalidReturnType: src/Mapping/ClassMetadata.php#L573
src/Mapping/ClassMetadata.php:573:22: InvalidReturnType: The declared return type 'array<array-key, ReflectionProperty|null>' for Doctrine\ORM\Mapping\ClassMetadata::getReflectionProperties is incorrect, got 'Doctrine\ORM\Mapping\LegacyReflectionFields<string, ReflectionProperty>|array<string, ReflectionProperty>' (see https://psalm.dev/011)
InvalidReturnStatement: src/Mapping/ClassMetadata.php#L577
src/Mapping/ClassMetadata.php:577:16: InvalidReturnStatement: The inferred type 'Doctrine\ORM\Mapping\LegacyReflectionFields<string, ReflectionProperty>|array<string, ReflectionProperty>' does not match the declared return type 'array<array-key, ReflectionProperty|null>' for Doctrine\ORM\Mapping\ClassMetadata::getReflectionProperties (see https://psalm.dev/128)
MissingTemplateParam: src/Mapping/LegacyReflectionFields.php#L21
src/Mapping/LegacyReflectionFields.php:21:41: MissingTemplateParam: Doctrine\ORM\Mapping\LegacyReflectionFields has missing template params when extending ArrayAccess, expecting 2 (see https://psalm.dev/182)
MissingTemplateParam: src/Mapping/LegacyReflectionFields.php#L21
src/Mapping/LegacyReflectionFields.php:21:54: MissingTemplateParam: Doctrine\ORM\Mapping\LegacyReflectionFields has missing template params when extending IteratorAggregate, expecting 2 (see https://psalm.dev/182)
InvalidPropertyAssignmentValue: src/Mapping/LegacyReflectionFields.php#L69
src/Mapping/LegacyReflectionFields.php:69:13: InvalidPropertyAssignmentValue: $this->reflFields with declared type 'array<string, ReflectionProperty>' cannot be assigned type 'non-empty-array<string, ReflectionProperty|null>' (see https://psalm.dev/145)
ArgumentTypeCoercion: src/Mapping/LegacyReflectionFields.php#L69
src/Mapping/LegacyReflectionFields.php:69:70: ArgumentTypeCoercion: Argument 1 of Doctrine\ORM\Mapping\LegacyReflectionFields::getAccessibleProperty expects class-string, but parent type string provided (see https://psalm.dev/193)
PossiblyNullArgument: src/Mapping/LegacyReflectionFields.php#L69
src/Mapping/LegacyReflectionFields.php:69:82: PossiblyNullArgument: Argument 2 of Doctrine\ORM\Mapping\LegacyReflectionFields::getAccessibleProperty cannot be null, possibly null value provided (see https://psalm.dev/078)
InvalidPropertyAssignmentValue: src/Mapping/LegacyReflectionFields.php#L73
src/Mapping/LegacyReflectionFields.php:73:21: InvalidPropertyAssignmentValue: $this->reflFields with declared type 'array<string, ReflectionProperty>' cannot be assigned type 'non-empty-array<string, ReflectionProperty|null>' (see https://psalm.dev/145)
Static Analysis with PHPStan (3.8.2, phpstan-dbal3.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/
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/