Problem/Motivation

The namespace of EmbedCKEditor5PluginBase is namespace Drupal\entity_embed\Plugin\CKEditor5Plugin; when it s folder structure is /embed/src/Plugin/CKEditor5Plugin/EmbedCKEditor5PluginBase.php

This makes our pipeline complaining about Class Drupal\entity_embed\Plugin\CKEditor5Plugin\EmbedCKEditor5PluginBase located in ./html/modules/contrib/embed/src/Plugin/CKEditor5Plugin/EmbedCKEditor5PluginBase.php does not comply with psr-4 autoloading standard (rule: Drupal\embed\ => ./html/modules/contrib/embed/src). Skipping.

I believe it s related to our update toward slevomat/coding-standard 8.16.2

Steps to reproduce

Proposed resolution

Update the namespace of EmbedCKEditor5PluginBase to follow PR-4 standard.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 embed--2025-06-30--3517882--mr-26.patch607 bytesrajab natshah

Issue fork embed-3517882

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

uber_denis created an issue. See original summary.

uber_denis opened merge request !26

uber_denis’s picture

uber_denis commented
Status: Active » Needs review
rajab natshah’s picture

rajab natshah
he/him
commented

Facing the same issue. thank you for the MR
Hoping for a 2.0.0 tag release ( to manage old and new sites )

rajab natshah’s picture

rajab natshah
he/him
commented
StatusFileSize
new embed--2025-06-30--3517882--mr-26.patch607 bytes

Attached a static embed--2025-06-30--3517882--mr-26.patch file, to this point for MR26.
To be used Composer Patches

nicodh’s picture

nicodh commented

Works for me on 2 sites!
Thanks!

nicodh’s picture

nicodh commented
Status: Needs review » Reviewed & tested by the community
ericgsmith’s picture

ericgsmith commented
Priority: Normal » Critical

++ please lets get this important fix in - is slowing down progress of #3531672: Drupal 10.5/11.2 Compatibility (tooltip broken, cannot edit embedded entities) as the tests there are failing due to this error

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Reviewed & tested by the community » Fixed

Merged into 8.x-1.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

rajab natshah’s picture

rajab natshah
he/him
commented

Thank you :) , for the merged committed fix.
Hopping for a release.

fago’s picture

fago
Primary language German
Location Vienna
commented

I ran into this once again. Could we have a new release with this fix? it's still un-fixed in latest release.