Warning: Undefined variable $content_types

Created on 7 November 2023, 8 months ago

Updated 20 January 2024, 5 months ago

Problem/Motivation

There is a potentially undefined variable in the ConditionalMessage entity when no content types exist yet. This can occur for instance when tests are running in a clean drupal environment (without content types).

Steps to reproduce

Run drupal without content types. The following warning will be generated: Undefined variable $content_types ConditionalMessage.php:383. The variable $content_types is set within a foreach loop, which potentially doesn't loop, hence the variable is not created.

Proposed resolution

Set the $content_types variable outside the foreach loop in ConditionalMessage.php:351.

Remaining tasks

Set the $content_types variable outside the foreach loop in ConditionalMessage.php:351.

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 ago8:43am 7 November 2023
Comment 8 months ago →
🇧🇪Belgium Fernly
Patch against 2.x dev branch.
Comment 6 months ago →
wranvaud
Merged! Thank you!
Status changed to Fixed 6 months ago5:07pm 6 January 2024
Comment 6 months ago →
System Message

wranvaud → committed 050d8aa9 on 2.x authored by Fernly →
Issue #3399779 by Fernly: Warning: Undefined variable $content_types
Comment 5 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