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 11 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