- Issue created by @drwilliamhobbs
- First commit to issue fork.
- πΊπΈUnited States mark_fullmer Tucson
Based on the analysis below and functional testing, this should be ready to merge and a new 8.x-1.7 release provided with Drupal 11 compatibility
Auditor checklist
[x] Deprecated Drupal code is remediated -- no deprecations found using Rector or Upgrade Status, or
[x] Deprecated PHP code is remediated -- change, noted below.
[x] Custom code is compatible with jQuery 4 -- none present
[x] Custom code core_version_requirement indicates Drupal 11 compatibility
[x] This module's composer.json file does not require any changes for Drupal 11 compatibility
[x] I updated the GitlabCI file to run Functional tests against both Drupal 10 and Drupal 11.
[x] Existing functional tests pass on Drupal 10 and Drupal 11Drupal-Rector Audit of Drupal deprecations
1 file with changes
===================1) web/modules/contrib/protected_pages/src/EventSubscriber/ProtectedPagesSubscriber.php:134 ---------- begin diff ---------- @@ @@ /** * {@inheritdoc} */ - public static function getSubscribedEvents() { + public static function getSubscribedEvents(): array { $events[KernelEvents::RESPONSE][] = ['checkProtectedPage']; return $events; } ----------- end diff ----------- Applied rules: * ProtectedStaticModulesPropertyRector * AddReturnTypeDeclarationRector [OK] 1 file would have been changed (dry-run) by Rector Audit of deprecated PHP <8.3 calls If no errors are listed below, php-compatibility did not find any. .......... 10 / 10 (100%)