Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
This module needs to be upgraded to allow for Drupal 11 compatibility
For sure fails with both changes needed to .info and from PHPStan
$ php vendor/bin/phpstan analyze $_WEB_ROOT/modules/custom/$CI_PROJECT_NAME $PHPSTAN_CONFIGURATION --no-progress --error-format=gitlab > phpstan-quality-report.json || true
$ php vendor/bin/phpstan analyze $_WEB_ROOT/modules/custom/$CI_PROJECT_NAME $PHPSTAN_CONFIGURATION --no-progress || EXIT_CODE=$?
------ ----------------------------------------------------------------------
Line src/Form/SettingsForm.php
------ ----------------------------------------------------------------------
44 Method Drupal\Core\Form\ConfigFormBase::__construct() invoked with 1
parameter, 2 required.
------ ----------------------------------------------------------------------
[ERROR] Found 1 error
See this change record https://www.drupal.org/node/3404140
Pretty sure we can get away without overloading the constructor by using $this->container()->get('entity_type.manager') down where we are looking for it in buildForm.
Issue fork paragraphs_report-3472669
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
generalredneckComment #4
generalredneckSo container() is a private function, so I couldn't access it. That said, I had to increase the minimum requirements for this piece for Drupal 10.2. See the Change Record linked in the description.
I've tested this in both Drupal 10.3 and 11.0 and there were no problems. Gitlab_ci passed.
Comment #6
generalredneck