Simplify ckeditor5_module_implements_alter() in favor of checking for ckeditor5 in media module

Created on 10 February 2025, 4 months ago

Problem/Motivation

ckeditor5_module_implements_alter() is a very complex workaround due to its hook needing to account for two different other hooks that are actually different hooks but called at the same time due to ModuleHandler::alter()'s capability to invoke multiple alter hooks at once.

Steps to reproduce

Proposed resolution

I think it simplifies things if we let Media check for ckeditor5 now, both are now stable modules, there isn't really any preference in either direction. And the logic is easier to understand if media conditionally adds the validation instead of adding it and then ckeditor5 removes it again.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Active

Version

11.0 πŸ”₯

Component

ckeditor5.module

Created by

πŸ‡¨πŸ‡­Switzerland berdir Switzerland

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