[D7 PHP 8.1] mb_strlen(): Passing null to parameter #1 ($string) of type string is deprecated in drupal_strlen()

Created on 24 October 2023, about 1 year ago

Updated 16 November 2023, about 1 year ago

Problem/Motivation

We are experiencing multiple deprecation warnings on PHP 8.1 in the drupal_strlen() function:

Deprecated function: mb_strlen(): Passing null to parameter #1 ($string) of type string is deprecated in drupal_strlen() (line 482 of includes/unicode.inc).

Steps to reproduce

Save a select_or_other → field in the field settings UI. This field's main $elements array when inside _form_validate() has 2 sub elements that have the #value as not null, but in that function it runs
elseif (isset($elements['#maxlength']) && drupal_strlen($elements['#value']) > $elements['#maxlength']) { without ensuring a string value.

Proposed resolution

Cast the input as a string.

🐛 Bug report

Status

Closed: duplicate

Version

7.0 ⚰️

Component

Base →

Last updated about 11 hours ago

Maintained by
🇺🇸United States @effulgentsia
🇬🇧United Kingdom @catch
🇬🇧United Kingdom @alexpott
🇦🇺Australia @larowlan
🇺🇸United States @bnjmnm
🇫🇷France @nod_
🇪🇸Spain @ckrina
🇬🇧United Kingdom @justafish

Created by

🇨🇦Canada joelpittet Vancouver

Live updates comments and jobs are added and updated live.

PHP 8.1
The issue particularly affects sites running on PHP version 8.1.0 or later.

Comments & Activities

Issue created by @joelpittet
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.1 & MySQL 5.7
last update about 1 year ago
2,161 pass
@joelpittet opened merge request.
Status changed to Closed: duplicate about 1 year ago7:12am 25 October 2023
Comment about 1 year ago →
🇸🇰Slovakia poker10
Thanks for reporting this! I think this is a duplicate of: 🐛 _form_validate sends null to drupal_strlen triggering deprecation notice Needs work . We also have a test in the mentioned issue, but it needs some little work to remove the extra failures, but otherwise I think it is pretty ready.
Comment about 1 year ago →
🇨🇦Canada joelpittet Vancouver
@poker10 Thanks yeah likely a dup but note this solution will resolve other similar bugs instead of playing whack-a-mole on each instance.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024