Fix all phpstan-drupal reports

Issue created by @fgm
Status changed to Needs review 12 months ago6:58am 7 July 2023
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 12 months ago
66 pass
Comment 12 months ago →
🇫🇷France fgm Paris, France
Up to level 2 included.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 12 months ago
66 pass
Comment 12 months ago →
🇫🇷France fgm Paris, France
Up to level 5 included.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 12 months ago
66 pass
Comment 12 months ago →
🇫🇷France fgm Paris, France
Up to level 6 included.
Comment 12 months ago →
System Message

fgm → committed 40af4081 on 8.x-1.x
Issue #3373060 by fgm: fix PHPstan reports.
Status changed to Fixed 12 months ago9:24am 7 July 2023
Comment 12 months ago →
🇫🇷France fgm Paris, France
Merged to current dev HEAD.
Status changed to Needs work 12 months ago10:25am 7 July 2023
Comment 12 months ago →
🇫🇷France fgm Paris, France
Actually, only the tests were fixed. More work needed.
Status changed to Needs review 12 months ago10:24pm 8 July 2023
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 12 months ago
66 pass
Comment 12 months ago →
🇫🇷France fgm Paris, France
Everything passing locally, but now phpcs complains that one should not redocument params.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 12 months ago
66 pass
Comment 12 months ago →
🇫🇷France fgm Paris, France
Tip: using @phpstan-param and @phpstan-return fixes it locally.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 12 months ago
Composer require failure
Comment 12 months ago →
🇫🇷France fgm Paris, France
Requiring coder 8.x-3.20 to avoid the silly return type warnings with 8.3.16.
Status changed to Needs work 12 months ago9:50pm 11 July 2023
Comment 12 months ago →
🇫🇷France fgm Paris, France
Head now passes at level 6. On to the next levels.
@fgm opened merge request.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 12 months ago
66 pass
Comment 12 months ago →
System Message

fgm → committed 6c4934b0 on 8.x-1.x
Resolve #3373060 "Phpstan"
Comment 12 months ago →
🇫🇷France fgm Paris, France
Status changed to Needs review 11 months ago10:57am 22 July 2023
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 11 months ago
74 pass
Comment 11 months ago →
🇫🇷France fgm Paris, France
Fixed up to level 8.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 11 months ago
66 pass
@fgm opened merge request.
Comment 11 months ago →
System Message
fgm → closed merge request !6
Comment 11 months ago →
System Message

fgm → committed 94ad03d9 on 8.x-1.x
Issue #3373060 by fgm: Fix all phpstan-drupal reports
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update 11 months ago
74 pass
Comment 11 months ago →
🇫🇷France fgm Paris, France
Should be complete.
Comment 11 months ago →
System Message

fgm → committed 48b89654 on 8.x-1.x
Issue #3373060 by fgm: Fix all phpstan-drupal reports
Status changed to Fixed 11 months ago2:00pm 22 July 2023
Comment 11 months ago →
🇫🇷France fgm Paris, France
Merged to current dev HEAD.
Comment 11 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Fix all phpstan-drupal reports

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments & Activities