Problem/Motivation

The above deprecated error message is triggered under PHP 8.1+.

Steps to reproduce

Just enable environment_indicator 4.0.x under PHP 8.1+. Do not set the "environment_indicator.current_release" setting. (The recommended way to add information about your release is to set the "environment_indicator.current_release" state. Use your git hooks to set the state using drush: drush sset environment_indicator.current_release v1.2.44).

Proposed resolution

Replace by !empty() the strlen() calls used just to check if length > 0.

Remaining tasks

Provide a patch.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 3260279-deprecated_function_strlen-2.patch643 bytesinterdruper

Comments

interdruper created an issue. See original summary.

interdruper’s picture

interdruper commented
Status: Active » Needs review
StatusFileSize
new 3260279-deprecated_function_strlen-2.patch643 bytes

Patch attached.

bohus ulrych’s picture

bohus ulrych
Location Pilsen (Czechia)
commented

Thanks. It works well with 4.0.4

bobooon’s picture

bobooon commented

Confirmed #2 resolves the PHP warnings.

siliconmeadow’s picture

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

Third confirmation that #2 resolves the PHP warning re: strlen() deprecation.

  • e0ipso committed 6de5964 on 4.x authored by interdruper 
    Issue #3260279 by interdruper, Bohus Ulrych, robphillips, siliconmeadow...
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Status: Reviewed & tested by the community » Fixed

Merged. Thanks everyone!

Status: Fixed » Closed (fixed)

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