Support for PHP 8.1 <

Open on Drupal.org →

Created on 23 January 2023, almost 2 years ago

Updated 21 November 2023, about 1 year ago

Problem/Motivation

There is a depricated usage of mb_strtolower(), we need to ensure that value passed to mb_strtolower() is not NULL

Notice visible in logs: "Deprecated function: mb_strtolower(): Passing null to parameter #1 (closed) ($string) of type string is deprecated in Drupal\enforce_user_fields\EventSubscriber\EnforceUserFieldsSubscriber->checkForUserFields() (line 109 of /var/www/modules/contrib/enforce_user_fields/src/EventSubscriber/EnforceUserFieldsSubscriber.php)"

Steps to reproduce

Run Drupal Core 9.4 < and PHP 8.1 <, enable module check message in logs.

Proposed resolution

Fix the conditions to check that value is not NULL

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Fixed

Version

2.0

Component

Code

Created by

🇵🇱Poland henk

Live updates comments and jobs are added and updated live.

ContributionWeekend2023

Sign in to follow issues

Comments & Activities

Issue created by @henk
Comment almost 2 years ago →
🇵🇱Poland henk
Patch with checking if config('whitelist') is not NULL in attachment.
Status changed to Needs review almost 2 years ago3:26pm 29 January 2023
Comment almost 2 years ago →
🇵🇱Poland henk
Comment almost 2 years ago →
🇵🇱Poland henk
Comment about 1 year ago →
🇪🇸Spain julien.reulos
Patch is applied correctly and fixes the issue, thanks!
Status changed to RTBC about 1 year ago1:23pm 3 November 2023
Comment about 1 year ago →
🇪🇸Spain julien.reulos
Status changed to Fixed about 1 year ago9:05pm 21 November 2023
Comment about 1 year ago →
🇺🇦Ukraine cosolom
Thanks all. Updated in dev. Will be released shortly
Comment about 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024