Contrib.social

Add validation constraints to system.file

Open on Drupal.org →
Created on 26 March 2024, 2 months ago
Updated 2 May 2024, about 1 month ago

Problem/Motivation

System module settings has 2 property paths that are not yet validatable:

vendor/bin/drush config:inspect --filter-keys=system.file --detail --list-constraints
➜  🤖 Analyzing…

 Legend for Data: 
  ✅❓  → Correct primitive type, detailed validation impossible.
  ✅✅  → Correct primitive type, passed all validation constraints.
 ---------------------------------------- --------- ------------- ------ ------------------------------------------ 
  Key                                      Status    Validatable   Data   Validation constraints                    
 ---------------------------------------- --------- ------------- ------ ------------------------------------------ 
  system.file                              Correct   67%           ✅❓   ValidKeys: '<infer>'                      
   system.file:                            Correct   Validatable   ✅✅   ValidKeys: '<infer>'                      
   system.file:_core                       Correct   Validatable   ✅✅   ValidKeys:                                
                                                                            - default_config_hash                   
   system.file:_core.default_config_hash   Correct   Validatable   ✅✅   NotNull: {  }                             
                                                                          Regex: '/^[a-zA-Z0-9\-_]+$/'              
                                                                          Length: 43                                
                                                                          ↣ PrimitiveType: {  }                     
   system.file:allow_insecure_uploads      Correct   Validatable   ✅✅   ↣ PrimitiveType: {  }                     
   system.file:default_scheme              Correct   NOT           ✅❓   ⚠️  @todo Add validation constraints here  
   system.file:temporary_maximum_age       Correct   NOT           ✅❓   ⚠️  @todo Add validation constraints here

Steps to reproduce

Proposed resolution

Add validation constraints to:

  1. system.file:default_scheme
  2. system.file:temporary_maximum_age

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

📌 Task
Status

Needs work

Version

11.0 🔥

Component
System  →

Last updated 1 day ago

No maintainer
Created by

🇮🇳India srishtiiee

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

Merge Requests

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024