Validate inputs of TypeResolver::resolveExpression(): only allow %parent, %type and %key

Created on 10 October 2023, about 1 year ago
Updated 8 April 2024, 9 months ago

Problem/Motivation

See #3382581-16: Add new `EntityBundleExists` constraint β†’ , -18 and -19:

While "gracefully returning" sounds good, it also means that the developer who writes a typo like EntityBundleExists: '%paren.target_entity_type_id', they get no feedback!

Because if we don't, we could probably throw an error and tell people something like: %paren found in resolved schema, there is a typo in your schema definition..

Steps to reproduce

Create a config schema definition with a typo in it. Note the absence of a useful error, making for a very painful DX.

Proposed resolution

Add helpful error message! Improved DX πŸš€

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

N/A

πŸ“Œ Task
Status

Fixed

Version

10.3 ✨

Component
Configuration  β†’

Last updated about 22 hours ago

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