InvalidArgumentException: Property label with value Imce Link does not exist on Drupal\ckeditor5\Plugin\CKEditor5PluginDefinition. in Drupal\ckeditor5\Plugin\CKEditor5PluginDefinition->__construct() (line 52 of docroot/core/modules/ckeditor5/src/Plugin/CKEditor5PluginDefinition.php).

CommentFileSizeAuthor
#7 3324281-7.patch1.69 KBbalintpekker

Issue fork imce-3324281

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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Rajeshreeputra created an issue. See original summary.

vipin.mittal18’s picture

vipin.mittal18
He/Him
CreditAttribution: vipin.mittal18 as a volunteer and at Acquia commented
Status: Active » Needs work

Unable to edit the text format on Drupal 9.5.0-rc1 due to following error:

InvalidArgumentException: Property label with value Imce Image does not exist on Drupal\ckeditor5\Plugin\CKEditor5PluginDefinition. in Drupal\ckeditor5\Plugin\CKEditor5PluginDefinition->__construct() (line 52 of /home/ide/project/docroot/core/modules/ckeditor5/src/Plugin/CKEditor5PluginDefinition.php).

Rajeshreeputra’s picture

Rajeshreeputra
Location Pune
CreditAttribution: Rajeshreeputra as a volunteer and at Acquia, Shivswarajya Inc. commented
Related issues: +#3314770: Enforce order of CKEditor 5 plugin settings in config export (as well as other sequences)

balintpekker made their first commit to this issue’s fork.

balintpekker’s picture

balintpekker
he/him
Primary language Hungarian
Location Budapest, Hungary
CreditAttribution: balintpekker at Cheppers for Acquia commented
Assigned: Unassigned » balintpekker

balintpekker opened merge request !13

balintpekker’s picture

balintpekker
he/him
Primary language Hungarian
Location Budapest, Hungary
CreditAttribution: balintpekker at Cheppers for Acquia commented
FileSize
3324281-7.patch1.69 KB

The label property is unnecessary in @CKEditor5Plugin annotations as they are declared below in the 'drupal' key, correctly. I'm uploading the diff of the merge request as a patch, too.

Edit: Couldn't add test to the 3.x-dev version, the patch will fail to apply on 8.x-1.x and 8.x-2.x. Tested locally on D9.5 and D10, removing the label key does not break D10 version, however it fixes D9.

balintpekker’s picture

balintpekker
he/him
Primary language Hungarian
Location Budapest, Hungary
CreditAttribution: balintpekker at Cheppers for Acquia commented
Assigned: balintpekker » Unassigned
Status: Needs work » Needs review
Rajeshreeputra’s picture

Rajeshreeputra
Location Pune
CreditAttribution: Rajeshreeputra as a volunteer and at Acquia, Shivswarajya Inc. commented
Status: Needs review » Reviewed & tested by the community

It worked as expected.

vipin.mittal18’s picture

vipin.mittal18
He/Him
CreditAttribution: vipin.mittal18 as a volunteer and at Acquia commented

@Balint Pekker: Yes we are able to continue on text formats on 9.5 RC version as well. Patch works. Thanks very much!!

RTBC+1

  • ufku committed 2995566 on 3.x authored by balintpekker 
    Issue #3324281: Removed unnecessary label property from plugin...
ufku’s picture

ufku CreditAttribution: ufku commented
Version: 3.0.3 » 3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed. Thanks

Status: Fixed » Closed (fixed)

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