Problem/Motivation

The following error appears in our logs:

Deprecated function: strlen(): Passing null to parameter #1 ($string) of type string is deprecated in Drupal\glossify\GlossifyBase->parseTooltipMatch() (Line 153 in /src/GlossifyBase.php)

this should be fixed.

Steps to reproduce

Proposed resolution

Fix the issue.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork glossify-3523609

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

grevil created an issue. See original summary.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Thanks, yes should be investigated and fixed, please.

grevil’s picture

grevil commented
Status: Active » Postponed (maintainer needs more info)

This shouldn't happen. "ignore_tags" has their default setting defined in the Doctrine Annotation above each filter plugin. So "ignore_tags" should always resolve at least to an empty string, not NULL. Maybe a simple clear all caches will fix the issue?

There is no need for an update hook, since this is a plugin. Filters never implement "defaultConfiguration()" for some reason. Not here nor in core. They are always defined in the doctrine annotation / PHP Attributes.

grevil opened merge request !49

grevil’s picture

grevil commented
Status: Postponed (maintainer needs more info) » Fixed

Talked with @anybody internally. Not really worth taking a deeper look at this.

  • grevil committed 7b883386 on 3.x 
    Issue #3523609: Deprecated function: strlen(): Passing null to parameter...

Status: Fixed » Closed (fixed)

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