Skip to content

Commit 90851bc

Browse files
rosierrosier
committed
Upgrade to PHPStan 2
1 parent 3277fc2 commit 90851bc

File tree

9 files changed

+64
-79
lines changed

9 files changed

+64
-79
lines changed

‎.github/workflows/lint.yaml‎

Lines changed: 22 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,11 +56,8 @@ jobs:
5656
restore-keys: ${{ runner.os }}-composer-
5757

5858
- name: "Install dependencies"
59-
run: composer install --ansi --no-interaction --no-progress
60-
61-
- name: Install PHPUnit
6259
id: install
63-
run: vendor/bin/simple-phpunit install
60+
run: composer install --ansi --no-interaction --no-progress
6461

6562
- name: Lint YAML files
6663
if: always() && steps.install.outcome == 'success'
@@ -94,6 +91,25 @@ jobs:
9491
if: always() && steps.install.outcome == 'success'
9592
run: composer audit
9693

94+
static-analysis:
95+
name: PHPStan
96+
runs-on: ubuntu-latest
97+
98+
steps:
99+
- name: Checkout code
100+
uses: actions/checkout@v4
101+
102+
- name: Setup PHP
103+
uses: shivammathur/setup-php@v2
104+
with:
105+
coverage: none
106+
php-version: '8.3'
107+
108+
- name: Install dependencies
109+
run: composer install --ansi --no-interaction --no-progress
110+
111+
- name: Install PHPUnit
112+
run: vendor/bin/simple-phpunit install
113+
97114
- name: Run PHPStan
98-
if: always() && steps.install.outcome == 'success'
99-
run: ./vendor/bin/phpstan analyze
115+
run: vendor/bin/phpstan analyze --no-progress -v

‎composer.json‎

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,9 @@
5454
"require-dev": {
5555
"dama/doctrine-test-bundle": "^8.0.2",
5656
"doctrine/doctrine-fixtures-bundle": "^3.5",
57-
"phpstan/extension-installer": "^1.2",
58-
"phpstan/phpstan": "^1.2",
59-
"phpstan/phpstan-doctrine": "^1.3",
60-
"phpstan/phpstan-symfony": "^1.2",
57+
"phpstan/phpstan": "^2.0",
58+
"phpstan/phpstan-doctrine": "^2.0",
59+
"phpstan/phpstan-symfony": "^2.0",
6160
"symfony/browser-kit": "^7",
6261
"symfony/css-selector": "^7",
6362
"symfony/debug-bundle": "^7",
@@ -70,8 +69,7 @@
7069
"config": {
7170
"allow-plugins": {
7271
"symfony/flex": true,
73-
"symfony/runtime": true,
74-
"phpstan/extension-installer": true
72+
"symfony/runtime": true
7573
},
7674
"platform": {
7775
"php": "8.2.0"

‎phpstan-baseline.neon‎

Lines changed: 21 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -1,71 +1,43 @@
11
parameters:
22
ignoreErrors:
33
-
4-
message: "#^Property App\\\\Entity\\\\Comment\\:\\:\\$author type mapping mismatch\\: property can contain App\\\\Entity\\\\User\\|null but database expects App\\\\Entity\\\\User\\.$#"
4+
message: '#^Class App\\Form\\ChangePasswordType extends generic class Symfony\\Component\\Form\\AbstractType but does not specify its types\: TData$#'
5+
identifier: missingType.generics
56
count: 1
6-
path: src/Entity/Comment.php
7+
path: src/Form/ChangePasswordType.php
78

89
-
9-
message: "#^Property App\\\\Entity\\\\Comment\\:\\:\\$content type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
10+
message: '#^Class App\\Form\\CommentType extends generic class Symfony\\Component\\Form\\AbstractType but does not specify its types\: TData$#'
11+
identifier: missingType.generics
1012
count: 1
11-
path: src/Entity/Comment.php
13+
path: src/Form/CommentType.php
1214

1315
-
14-
message: "#^Property App\\\\Entity\\\\Comment\\:\\:\\$post type mapping mismatch\\: property can contain App\\\\Entity\\\\Post\\|null but database expects App\\\\Entity\\\\Post\\.$#"
16+
message: '#^Class App\\Form\\PostType extends generic class Symfony\\Component\\Form\\AbstractType but does not specify its types\: TData$#'
17+
identifier: missingType.generics
1518
count: 1
16-
path: src/Entity/Comment.php
19+
path: src/Form/PostType.php
1720

1821
-
19-
message: "#^Property App\\\\Entity\\\\Post\\:\\:\\$author type mapping mismatch\\: property can contain App\\\\Entity\\\\User\\|null but database expects App\\\\Entity\\\\User\\.$#"
22+
message: '#^Class App\\Form\\Type\\DateTimePickerType extends generic class Symfony\\Component\\Form\\AbstractType but does not specify its types\: TData$#'
23+
identifier: missingType.generics
2024
count: 1
21-
path: src/Entity/Post.php
25+
path: src/Form/Type/DateTimePickerType.php
2226

2327
-
24-
message: "#^Property App\\\\Entity\\\\Post\\:\\:\\$content type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
28+
message: '#^Class App\\Form\\Type\\TagsInputType extends generic class Symfony\\Component\\Form\\AbstractType but does not specify its types\: TData$#'
29+
identifier: missingType.generics
2530
count: 1
26-
path: src/Entity/Post.php
31+
path: src/Form/Type/TagsInputType.php
2732

2833
-
29-
message: "#^Property App\\\\Entity\\\\Post\\:\\:\\$slug type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
34+
message: '#^Class App\\Form\\UserType extends generic class Symfony\\Component\\Form\\AbstractType but does not specify its types\: TData$#'
35+
identifier: missingType.generics
3036
count: 1
31-
path: src/Entity/Post.php
37+
path: src/Form/UserType.php
3238

3339
-
34-
message: "#^Property App\\\\Entity\\\\Post\\:\\:\\$summary type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
40+
message: '#^Call to function method_exists\(\) with ''Symfony\\\\Component\\\\Dotenv\\\\Dotenv'' and ''bootEnv'' will always evaluate to true\.$#'
41+
identifier: function.alreadyNarrowedType
3542
count: 1
36-
path: src/Entity/Post.php
37-
38-
-
39-
message: "#^Property App\\\\Entity\\\\Post\\:\\:\\$title type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
40-
count: 1
41-
path: src/Entity/Post.php
42-
43-
-
44-
message: "#^Property App\\\\Entity\\\\User\\:\\:\\$email type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
45-
count: 1
46-
path: src/Entity/User.php
47-
48-
-
49-
message: "#^Property App\\\\Entity\\\\User\\:\\:\\$fullName type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
50-
count: 1
51-
path: src/Entity/User.php
52-
53-
-
54-
message: "#^Property App\\\\Entity\\\\User\\:\\:\\$password type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
55-
count: 1
56-
path: src/Entity/User.php
57-
58-
-
59-
message: "#^Property App\\\\Entity\\\\User\\:\\:\\$username type mapping mismatch\\: property can contain string\\|null but database expects string\\.$#"
60-
count: 1
61-
path: src/Entity/User.php
62-
63-
-
64-
message: "#^Method App\\\\Entity\\\\User::getUserIdentifier\\(\\) should return non-empty-string but returns string\\.$#"
65-
count: 1
66-
path: src/Entity/User.php
67-
68-
-
69-
message: "#^Parameter \\#1 \\$function of class ReflectionFunction constructor expects Closure\\|string, callable\\(\\)\\: mixed given\\.$#"
70-
count: 1
71-
path: src/Twig/SourceCodeExtension.php
43+
path: tests/bootstrap.php

‎phpstan.dist.neon‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,23 @@
1+
includes:
2+
- phpstan-baseline.neon
3+
- vendor/phpstan/phpstan-doctrine/extension.neon
4+
- vendor/phpstan/phpstan-symfony/extension.neon
5+
- vendor/phpstan/phpstan-symfony/rules.neon
6+
17
parameters:
8+
bootstrapFiles:
9+
- vendor/bin/.phpunit/phpunit/vendor/autoload.php
10+
11+
doctrine:
12+
objectManagerLoader: tests/object-manager.php
13+
214
level: 6
315
paths:
416
- bin/
517
- config/
618
- public/
719
- src/
820
- tests/
21+
22+
symfony:
23+
containerXmlPath: var/cache/dev/App_KernelDevDebugContainer.xml

‎phpstan.neon.dist‎

Lines changed: 0 additions & 14 deletions
This file was deleted.

‎src/Form/Type/DateTimePickerType.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public function configureOptions(OptionsResolver $resolver): void
4949
]);
5050
}
5151

52-
public function getParent(): ?string
52+
public function getParent(): string
5353
{
5454
return DateTimeType::class;
5555
}

‎src/Form/Type/TagsInputType.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public function buildView(FormView $view, FormInterface $form, array $options):
5252
$view->vars['tags'] = $this->tags->findAll();
5353
}
5454

55-
public function getParent(): ?string
55+
public function getParent(): string
5656
{
5757
return TextType::class;
5858
}

‎tests/Controller/Admin/BlogControllerTest.php‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,6 @@ public function testAdminNewPost(): void
110110
/** @var PostRepository $postRepository */
111111
$postRepository = static::getContainer()->get(PostRepository::class);
112112

113-
/** @var \App\Entity\Post $post */
114113
$post = $postRepository->findOneByTitle($postTitle);
115114

116115
$this->assertNotNull($post);

‎tests/Controller/UserControllerTest.php‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@ public function testEditUser(): void
7575

7676
$this->assertResponseRedirects('/en/profile/edit', Response::HTTP_SEE_OTHER);
7777

78-
/** @var User $user */
7978
$user = $userRepository->findOneByEmail($newUserEmail);
8079

8180
$this->assertNotNull($user);

0 commit comments

Comments
 (0)