WithDisplayBuilderInterface follow-ups

Created on 26 July 2025, 3 months ago

Problem/Motivation

Following 📌 [1.0.0-beta1] Add proper page management Active , 📌 Align config storage properties & mechanisms Active and 📌 Implement WithDisplayBuilderInterface in Entity View Active , there are some cleanups to do:

  • Remove DisplayBuilder::DISPLAY_BUILDER_CONFIG
  • Rename EntityWithDisplayBuilderInterface to WithDisplayBuilderInterface because it is not only entities
  • Merge StorageProperties enum to static constant in ConfigFormBuilder?
  • Remaining config dependency management stuff?
  • Do we keep the \Drupal::service('page_cache_kill_switch')->trigger();?

Proposed resolution

Clean that.

📌 Task
Status

Active

Version

1.0

Component

Main / Misc.

Created by

🇫🇷France pdureau Paris

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

Merge Requests

Comments & Activities

  • Issue created by @pdureau
  • 🇫🇷France pdureau Paris

    Also, if we try to improve our config schema...

    From:

        display_builder:
          type: string
          label: "Used Display Builder"
    

    To something like that:

        display_builder:
          type: string
          label: "Used Display Builder"
          constraints:
            ConfigExists:
              prefix: 'display_builder.display_builder.'
    

    From:

        sources:
          type: ignore
          label: "Sources"

    To something like that:

        sources:
          type: sequence
          label: 'Sources'
          sequence:
            type: ui_patterns_source.[id]
            label: 'Source'

    extending DefaultLazyPluginCollection

    Is it unlocking new cool Drupal feature like automatic casting ? automatic dependency calculation ? so we can reduce the complexity of our code

  • 🇫🇷France pdureau Paris

    Remove DisplayBuilder::DISPLAY_BUILDER_CONFIG

    DONE

    Rename EntityWithDisplayBuilderInterface to WithDisplayBuilderInterface because it is not only entities

    DONE

    Merge StorageProperties enum to static constant in ConfigFormBuilder?

    DONE

    Is it unlocking new cool Drupal feature like automatic casting ? automatic dependency calculation ? so we can reduce the complexity of our code Is it unlocking new cool Drupal feature like automatic casting ? automatic dependency calculation ? so we can reduce the complexity of our code

    UNDER INVESTIGATION

    Remaining config dependency management stuff?

    Will be moved to its own beta issue.

    Do we keep the \Drupal::service('page_cache_kill_switch')->trigger();?

    Will be moved to its own beta issue.

  • First commit to issue fork.
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024