[Drupal 11] Require $field_definition to be passed to FieldItemInterface::fieldSettingsToConfigData()

Created on 3 August 2023, over 1 year ago
Updated 17 August 2023, over 1 year ago

Problem/Motivation

In 🐛 Field [storage] config have incomplete settings until they are saved Fixed , we decided to move certain field normalization work into \Drupal\Core\Field\FieldItemInterface::fieldSettingsToConfigData(). That necessitates adding an argument to an interface method, which is optional in Drupal 10, but required in Drupal 11.

In this issue, we should make that argument required.

Proposed resolution

Make the $field_definition argument of \Drupal\Core\Field\FieldItemInterface::fieldSettingsToConfigData() required, and adjust implementing classes accordingly.

API changes

Yes.

Release notes snippet

TBD?

📌 Task
Status

Closed: outdated

Version

11.0 🔥

Component
Field 

Last updated 1 day ago

Created by

🇺🇸United States phenaproxima Massachusetts

Live updates comments and jobs are added and updated live.
  • API change

    Changes an existing API or subsystem. Not backportable to earlier major versions, unless absolutely required to fix a critical bug.

  • API clean-up

    Refactors an existing API or subsystem for consistency, performance, modularization, flexibility, third-party integration, etc. May imply an API change. Frequently used during the Code Slush phase of the release cycle.

Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024