mb_strtolower passing null is deprecated

Created on 7 November 2023, 8 months ago

Updated 8 November 2023, 8 months ago

Problem/Motivation

This is an edge case, probably only occurring during automated tests.
When no plugin config is set, the RequestPathInclexcl:evaluate() method produces the following warning:
Deprecated: mb_strtolower(): Passing null to parameter #1 ($string) of type string is deprecated in web/modules/contrib/condition_path/src/Plugin/Condition/RequestPathInclexcl.php on line 90

Proposed resolution

Make sure NULL is not passed to mb_strtolower().

Remaining tasks

Make sure NULL is not passed to mb_strtolower().

User interface changes

None.

API changes

None.

Data model changes

None.

🐛 Bug report

Status

Fixed

Version

2.0

Component

Code

Created by

🇧🇪Belgium Fernly

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @Fernly
Status changed to Needs review 8 months ago11:48am 7 November 2023
Comment 8 months ago →
🇧🇪Belgium Fernly
Patch against 2.0.x branch.
Status changed to RTBC 8 months ago2:11pm 8 November 2023
Comment 8 months ago →
🇧🇪Belgium Fernly
Comment 8 months ago →
System Message

Fernly → committed 4aea12b2 on 2.0.x
Issue #3399820 by Fernly: mb_strtolower passing null is deprecated
Status changed to Fixed 8 months ago2:28pm 8 November 2023
Comment 8 months ago →
🇧🇪Belgium Fernly
Fixed in version 2.0.1.
Comment 7 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024