field can be saved with an empty display_id

Created on 3 July 2024, 8 months ago

Updated 11 October 2024, 4 months ago

If there is a restriction on the types of display that can be referenced in a field, it's possible to save broken data:

1. Select a view which doens't have any displays of that type
2. The display ID form element updates to show an empty list

The widget allows the field to be saved at this point.

🐛 Bug report

Status

Active

Version

3.0

Component

User interface

Created by

🇬🇧United Kingdom joachim

Live updates comments and jobs are added and updated live.

Merge Requests

Issue created by @joachim
First commit to issue fork.
Merge request !17Resolve #3458852 "Field can be" → (Open) created by danflanagan8
Comment 4 months ago →
🇺🇸United States danflanagan8 St. Louis, US
I put in an MR that uses #states to make the display id required when the view id is set. Maybe real validation would be better because #states requires JS, but seriously who is configuring this without JS enabled.

Production build 0.71.5 2024