Contrib.social

Configuration validation fails for email fields with token values

Open on Drupal.org →
Created on 20 December 2023, over 1 year ago
Updated 22 April 2024, about 1 year ago

Problem/Motivation

After upgrading to D10.2 I get errors from Configuration Inspector → module, saying, that ""[site:mail]" is not a valid email address." for fields like "default_to_mail". This happens, because in issue #3364506 ✨ Add optional validation constraint support to ConfigFormBase Fixed constraint has been added.

Steps to reproduce

Install D10.2, config_inspector and webform modules, create webform with email handler or save settings and in email fields, point token values in email fields. Then check configuration with config_inspector module.

Proposed resolution

I'm not sure about the best way, but as a starting point I suggest changing type from "email" to "string". It looks more appropriate, than changing core email validator, because it's our decision to support tokens as values for configuration.

Remaining tasks

Review patch, give feedback, fix patch.

🐛 Bug report
Status

Fixed

Version

6.2

Component

Code

Created by

🇷🇺Russia ilya.no

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

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024