Problem/Motivation

Error message
Deprecated function: htmlspecialchars(): Passing null to parameter #1 ($string) of type string is deprecated in check_plain() (line 1908 of /var/www/html/public/includes/bootstrap.inc).

Traced to originating from here
panels/plugins/display_renderers/panels_renderer_standard.class.php:447

Steps to reproduce

Enable panels, create a new panels page

Proposed resolution

Check or cast string before check_plain()

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork panels-3336591

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

joelpittet created an issue. See original summary.

joelpittet opened merge request !8

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Title: Deprecated function: htmlspecialchars(): Passing null to parameter #1 ($string) » PHP 8.2 Compatibility
Issue tags: +GlobalContributionWeekend2023
norman.lol’s picture

norman.lol
he/him
Primary language German
Location Berlin
commented
Issue tags: -GlobalContributionWeekend2023 +ContributionWeekend2023

Wrong tag

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Parent issue: » #3321791: Plan for Panels 7.x-3.12
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Related issues: +#3304807: [META] Make Drupal 7 compatible with PHP 8.2
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Issue tags: +PHP 8.2
Related issues: +#3337800: phpcs endless loop with "Drupal" rules due to missing @endcode

Fixed issue related to #3337800: phpcs endless loop with "Drupal" rules due to missing @endcode

norgef made their first commit to this issue’s fork.

norgef opened merge request !9

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

Thanks for moving over to this issue to help out @norgef, MR !8 has the same fixes already that are in MR !9, just need to fix the test failures

  • joelpittet committed 0815c56f on 7.x-3.x 
    Issue #3336591 by joelpittet, norgef, leymannx: PHP 8.2 Compatibility
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Fixed

Those tests failures aren't reproducible locally so I assume it's just testbot being strange

Status: Fixed » Closed (fixed)

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