Undefined property: DiffFormatter::$line_stats DiffEngine.php:861 with PHP 8

Created on 27 April 2023, over 1 year ago

Updated 5 May 2023, over 1 year ago

Problem/Motivation

Error when running drush fda with PHP 8:

Undefined property: DiffFormatter::$line_stats DiffEngine.php:861 [warning]
Undefined array key "counter" DiffEngine.php:861 [warning]
Undefined array key "x" DiffEngine.php:861 [warning]
Undefined array key "y" DiffEngine.php:864

Steps to reproduce

Install features and diff, create a feature with a diff, run drush fda.

Proposed resolution

Define $line_stat property in DiffFormatter class, as it is defined in DrupalDiffFormatter class. Features uses the DiffFormatter class directly.

🐛 Bug report

Status

RTBC

Version

3.4

Component

Code

Created by

🇺🇸United States lwalley

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @lwalley
Open on Drupal.org →
Core: 7.x + Environment: PHP 5.3 & MySQL 5.5
last update over 1 year ago
Waiting for branch to pass
@lwalley opened merge request.
Status changed to Needs review over 1 year ago8:27pm 27 April 2023
Comment over 1 year ago →
🇺🇸United States lwalley
Status changed to RTBC over 1 year ago1:51pm 5 May 2023
Comment over 1 year ago →
🇬🇧United Kingdom oldspot
After upgrading to PHP 8.1 and latest diff module version I got the exact same warnings.
Can confirm this patch fixes them and can no longer see them.
Thanks

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024