Problem/Motivation

phpcs detects

FILE: /var/www/html/web/modules/contrib/flexible_permissions/tests/src/Unit/ChainPermissionCalculatorTest.php
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 291 | WARNING | Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable
     |         | parameter: $variation_cache.
 292 | WARNING | Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable
     |         | parameter: $variation_cache_static.
 293 | WARNING | Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable
     |         | parameter: $cache_static.
 294 | WARNING | Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable
     |         | parameter: $account_switcher.
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Command icon 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:

Comments

abelass created an issue. See original summary.

danherrero’s picture

Status: Active » Reviewed & tested by the community

I observed the deprecation warnings reported here with flexible_permissions 2.0.1 installed in a PHP 8.4 environment.

I tested with MR !11 and can confirm that it resolves the warnings.

maursilveira’s picture

I second @danherrero, and although I now this module is minimally maintained, it'd be great to have a new release that contains this fix.

Thank you.

recrit’s picture

Attaching a static patch for composer builds.