Follow-up for #3382510: Throw \LogicException when >1 #config_target in the same form targets the same property path

Created on 13 November 2023, about 1 year ago
Updated 14 November 2023, about 1 year ago

Problem/Motivation

Oversight in πŸ“Œ Introduce a new #config_target Form API property to make it super simple to use validation constraints on simple config forms, and adopt it in several core config forms Fixed .

Surfaced by @alexpott in πŸ“Œ ConfigFormBase + validation constraints: support composite form elements Needs review at https://git.drupalcode.org/project/drupal/-/merge_requests/5240/diffs#no...

Steps to reproduce

Two #config_targets with the same targeted property path currently just overwrite each other 😬

Proposed resolution

Throw a \LogicException with a helpful message.

Remaining tasks

  1. βœ…
  2. βœ…
  3. Reviews.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

None.

πŸ› Bug report
Status

Fixed

Version

10.2 ✨

Component
ConfigurationΒ  β†’

Last updated 3 days ago

Created by

πŸ‡§πŸ‡ͺBelgium wim leers Ghent πŸ‡§πŸ‡ͺπŸ‡ͺπŸ‡Ί

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024