Problem/Motivation

There 2 PHP 8.4 deprecated errors in the module.

Steps to reproduce

❯ ~/workspace/php-cs-fixer/vendor/bin/php-cs-fixer check backend/web/modules/contrib/password_policy/ --rules=nullable_type_declaration_for_default_null_value

1) backend/web/modules/contrib/password_policy/password_policy_length/tests/src/Functional/PasswordLengthOperations.php
2) backend/web/modules/contrib/password_policy/src/Plugin/migrate/process/PolicyConfiguration.php

Found 2 of 60 files that can be fixed in 0.203 seconds, 26.50 MB memory used

Proposed resolution

Fix them.

Remaining tasks

Merge request.

User interface changes

none

API changes

none

Data model changes

none

CommentFileSizeAuthor
#3 password-policy-3516906.patch1.87 KBklausi

Issue fork password_policy-3516906

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:

About issue forks

Comments

klausi created an issue. See original summary.

klausi opened merge request !104

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Active » Needs review
StatusFileSize
new password-policy-3516906.patch1.87 KB

merge request created, uploading stable patch fiel for composer patches.

andriy khomych’s picture

andriy khomych commented
Status: Needs review » Reviewed & tested by the community

Thanks, Klausi, looks good to me.
Moved to RTBC.

vishalkhode made their first commit to this issue’s fork.

  • vishalkhode committed 999abf85 on 4.0.x authored by klausi 
    Issue #3516906 by klausi, vishalkhode: PHP 8.4 nullable types must be...
vishalkhode’s picture

vishalkhode commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

rajab natshah’s picture

rajab natshah
he/him
commented

Thank you for the fix.
Hoping for a release.

rajab natshah’s picture

rajab natshah
he/him
commented
Priority: Normal » Major
rajab natshah’s picture

rajab natshah
he/him
commented

Hopping for a tag release.
Thank you :)

jennypanighetti’s picture

jennypanighetti commented

Just found this as closed, but still no new release to include this??

When can we get a release so we can stop applying a patch?