Contrib.social

Stream wrappers not registered when installing module's default config

Open on Drupal.org →
Created on 24 January 2024, over 1 year ago

Problem/Motivation

Discovered in ✨ [PP-1] Mark parts of CKEditor 5 and Editor config schema as fully validatable Postponed .

That issue added validation for editor.editor.*:image_upload.scheme: that must be one of the registered StreamWrapperInterface::WRITE_VISIBLE stream wrappers.

However, it appears that in some scenarios, the public stream wrapper is present in \Drupal\Core\StreamWrapper\StreamWrapperManager::$info but absent from \Drupal\Core\StreamWrapper\StreamWrapperManager::$wrappers, which causes \Drupal\Core\StreamWrapper\StreamWrapperManager::getNames(StreamWrapperInterface::WRITE_VISIBLE) to return … nothing! 😱

Consequence: a validation error like this is generated:

The file storage you selected is not a visible, readable and writable stream wrapper. Possible choices: <em class="placeholder"></em>.

… because there are no possible choices 😅

Steps to reproduce

Remove 

    'editor.editor.*' => [
      'image_upload.scheme' => [
        '^The file storage you selected is not a visible, readable and writable stream wrapper\. Possible choices: <em class="placeholder"><\/em>\.$',
      ],
    ],

from \Drupal\Core\Config\Schema\SchemaCheckTrait::$ignoredPropertyPaths.

A number of tests will fail.

Proposed resolution

TBD

Merge request link

Remaining tasks

TBD

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

N/A

🐛 Bug report
Status

Active

Version

11.0 🔥

Component
Base  →

Last updated about 2 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

Not all content is available!

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

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024