Missing schema file

Created on 2 January 2023, over 1 year ago
Updated 13 April 2023, about 1 year ago

Problem/Motivation

The module doesn't have a schema file, so this issue appear after review the configuratio of the website :

drupal ckeditor_media_embed.settings No schema

πŸ› Bug report
Status

Needs work

Version

1.0

Component

Code

Created by

πŸ‡§πŸ‡΄Bolivia vacho Cochabamba

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ͺπŸ‡ΈSpain tunic Madrid

    I've recently hit the hard wall of the schema files, so I'm not really sure about the following review:

    1. +++ b/config/schema/ckeditor_media_embed.schema.yml
      @@ -0,0 +1,20 @@
      +  type: mapping
      

      Per https://www.drupal.org/docs/drupal-apis/configuration-api/configuration-... β†’ I think this should be configuration_object instead of mapping. On that section, at the end, it states:

      In all cases supported by core, the top level item in the .yml file will be a mapping with elements described in a mapping list underneath. You should use either of the two specific mapping subtypes config_object or config_entity.

    2. +++ b/config/schema/ckeditor_media_embed.schema.yml
      @@ -0,0 +1,20 @@
      +    langcode:
      +      type: string
      +      label: 'Langcode'
      +    _core:
      +      type: _core_config_info
      +      label: 'Core config info'
      
    3. Related to previous comment, langcode and _core are provided by the config_object.

    4. Additionally, the third_party_settings property is not declared in this schema file.

      An example of what it would look like:

      third_party_settings:
        oembed_lazyload:
          oembed_lazyload_enabled: 1
          oembed_lazyload_providers:
            YouTube: YouTube
            Vimeo: 0
      
Production build 0.69.0 2024