Create a new Exception type for a dynamic prop missing a host entity

Created on 17 July 2024, 7 months ago
Updated 18 September 2024, 4 months ago

Overview

Follow-up to 📌 Lift most logic out of ComponentTreeItem::preSave() and into a new validation constraint Needs work . see https://git.drupalcode.org/project/experience_builder/-/merge_requests/6...

in that issue we ignored and OutOfRangeException if the component tree was config because at that point we don't have a host entity.

Proposed resolution

  1. Make new exception type for when the host entity is missing.
  2. Throw that exception in \Drupal\experience_builder\PropSource\DynamicPropSource::evaluate instead of OutOfRangeException
  3. Catch that exception in ValidComponentTreeConstraintValidator instead of OutOfRangeException
📌 Task
Status

Needs work

Component

Page builder

Created by

🇺🇸United States tedbow Ithaca, NY, USA

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024