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

Created on 13 November 2023, over 1 year ago
Updated 14 November 2023, over 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 44 minutes 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