PHP 8 issue with entity_embed_field_info_formatter_settings

Created on 15 November 2023, 7 months ago

Updated 3 April 2024, 3 months ago

Problem/Motivation

Presently, entity_embed_field_info_formatter_settings() sometimes generates Optional parameter $settings declared before required parameter $form_state is implicitly treated as a required parameter in require_once() due to its function signature placing an optional parameter before required parameters which is deprecated as of PHP 8.0.0.

Steps to reproduce

Load entity_embed on a site using PHP 8+

Proposed resolution

Potentially drop declaration of settings parameter as optional, since all usages of it seem to supply the parameter.

API changes

Change $settings to required parameter?

🐛 Bug report

Status

Closed: outdated

Version

3.0

Component

Code

Created by

tadean

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @tadean
Comment 7 months ago →
tadean
Added potential patch.
Status changed to Needs review 7 months ago7:04pm 15 November 2023
Open on Drupal.org →
Core: 7.x + Environment: PHP 5.3 & MySQL 5.5
last update 7 months ago
Waiting for branch to pass
Comment 7 months ago →
tadean
Status changed to Closed: outdated 3 months ago6:53am 3 April 2024
Comment 3 months ago →
🇨🇦Canada Devin Carlson Sudbury, ON
Thanks for the patch!

This was fixed as part of adding alt + title support to display formatters.

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024