Add validation constraints to all system.* simple config (except system.rss)

Created on 21 February 2024, about 1 year ago

Problem/Motivation

Per 🌱 [meta] Add constraints to all simple configuration Active , the current state of validatable simple config in the System module is:

  • system.site
  • βœ…system.maintenance
  • system.cron
  • system.date
  • system.diff
  • system.logging
  • system.performance
  • system.rss
  • system.theme
  • system.file
  • system.image
  • system.image.gd
  • system.mail
  • system.advisories
  • βœ…system.feature_flags

All of these seem possible to make fully validatable. Sole exception: system.rss, because it A) depends on config entities, B) it depends on multiple or none at the same time. It probably makes more sense to remove that config πŸ˜…

P.S.: system.theme.global isn't in that list because it extends type: theme_settings, not type.config, but fortunately πŸ“Œ Add validation constraints to `type: theme_settings` Active will handle that πŸ‘

Proposed resolution

  1. Add validation constraints.
  2. Mark FullyValidatable.
  3. Use #config_target in config forms wherever possible

Remaining tasks

Review.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

N/A

πŸ“Œ Task
Status

Needs review

Version

11.0 πŸ”₯

Component
SystemΒ  β†’

Last updated about 2 hours ago

No maintainer
Created by

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

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

Merge Requests

Comments & Activities

Production build 0.71.5 2024