Deprecations PHP 8.4

Created on 24 December 2024, 3 months ago
Deprecated: Drupal\pathauto\AliasStorageHelper::__construct(): Implicitly marking parameter $entity_type_manager as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/contrib/pathauto/src/AliasStorageHelper.php on line 79

Deprecated: Drupal\pathauto\PathautoGenerator::__construct(): Implicitly marking parameter $alias_type_manager as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/contrib/pathauto/src/PathautoGenerator.php on line 1
🐛 Bug report
Status

Active

Version

1.0

Component

Code

Created by

🇮🇱Israel heyyo Jerusalem

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @heyyo
  • 🇺🇸United States pwarn Vermont

    I believe this is the same issue I'm having, but with a slightly different message

    Deprecated: Drupal\pathauto\Plugin\Action\UpdateAction::access(): Implicitly marking parameter $account as nullable is deprecated, the explicit nullable type must be used instead in ../web/modules/contrib/pathauto/src/Plugin/Action/UpdateAction.php on line 31

  • Merge request !92update changes → (Open) created by Unnamed author
  • Pipeline finished with Success
    3 months ago
    Total: 184s
    #378749
  • 🇮🇳India chandansha

    @niharika.s i have tested MR 92. there is still some issue remaining please find and resolve it thanks!!

    PHP Deprecated:  Drupal\pathauto\Commands\PathautoCommands::generateAliases(): Implicitly marking parameter $types as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/custom/pathauto-3495869/src/Commands/PathautoCommands.php on line 89
    
    Deprecated: Drupal\pathauto\Commands\PathautoCommands::generateAliases(): Implicitly marking parameter $types as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/custom/pathauto-3495869/src/Commands/PathautoCommands.php on line 89
    PHP Deprecated:  Drupal\pathauto\Commands\PathautoCommands::deleteAliases(): Implicitly marking parameter $types as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/custom/pathauto-3495869/src/Commands/PathautoCommands.php on line 135
    
    Deprecated: Drupal\pathauto\Commands\PathautoCommands::deleteAliases(): Implicitly marking parameter $types as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/custom/pathauto-3495869/src/Commands/PathautoCommands.php on line 135
    PHP Deprecated:  Drupal\pathauto\AliasStorageHelper::__construct(): Implicitly marking parameter $entity_type_manager as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/custom/pathauto-3495869/src/AliasStorageHelper.php on line 79
    
    Deprecated: Drupal\pathauto\AliasStorageHelper::__construct(): Implicitly marking parameter $entity_type_manager as nullable is deprecated, the explicit nullable type must be used instead in /var/www/html/web/modules/custom/pathauto-3495869/src/AliasStorageHelper.php on line 79
  • 🇮🇳India sayan_k_dutta

    Working on it.

  • 🇨🇭Switzerland berdir Switzerland

    We want to run tests against 8.4, at least temporarily:

    Add this to .gitlab-ci.yml:

    composer:
      extends: .composer-base
      variables:
        PHP_VERSION: '8.4'
    

    we could also opt in to MAX_PHP, but that's still set to 8.3, I want to create an issue to update it to 8.4, didn't do that yet.

  • Pipeline finished with Failed
    3 months ago
    Total: 279s
    #379591
  • Pipeline finished with Failed
    3 months ago
    Total: 192s
    #379600
  • 🇨🇭Switzerland berdir Switzerland

    Nice. the tests still fail in (https://git.drupalcode.org/issue/pathauto-3495869/-/jobs/3824204) but they only fail on forum integration tests, so we need an issue in the forum contrib project to make the necessary changes.

  • 🇮🇳India sayan_k_dutta

    I don't have the required knowledge for the failing tests. Please someone look into it.
    Unassigning myself.

  • 🇮🇳India chandansha

    I have tested MR 92. Now all deprecation error are resolved.
    I moved it to RTBC.
    Thanks!!

  • 🇮🇳India prem suthar Ahemdabad- Gujrat , Jodhpur - Rajsthan

    I've added the issue to the Forum Contrib project for the PHP 8.4 fixes as per point #9.

  • 🇳🇴Norway jonsimonsen

    Why are there two different issues here? Isn't this the same as this https://www.drupal.org/project/pathauto/issues/3489108 🐛 Fix PHP 8.4 implicit nullable deprecation Needs review ?

    I went with this one, and it seems to have at least gotten rid of all deprecation messages I saw when running drush commands.

  • Status changed to Closed: duplicate 30 days ago
  • 🇦🇹Austria klausi 🇦🇹 Vienna

    Closing this as duplicate of 🐛 Fix PHP 8.4 implicit nullable deprecation Needs review . Let's continue there!

Production build 0.71.5 2024