Problem/Motivation

Reported by @nod_ and @lauriii in #3248425: Ensure that all classes and functions in Drupal-specific CKEditor 5 plugins are documented 👍

There's some crufty conversions happening in core/modules/ckeditor5/js/ckeditor5_plugins/drupalImage/src/imageupload/drupalimageuploadadapter.js.

Proposed resolution

Use response.entity_type in DrupalImageUploadEditing.

Remaining tasks

TBD

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork drupal-3275237

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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers created an issue. See original summary.

Wim Leers credited lauriii.

Wim Leers credited nod_.

Wim Leers’s picture

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

Wim Leers’s picture

Status: Active » Needs work

The merge request looks delightfully simple! 🤩

Looks like this is crashing at least one functional JS test: CKEditor5Test 😅

Wim Leers’s picture

Status: Needs work » Reviewed & tested by the community

Looks ready now! 😄

lauriii’s picture

  • lauriii committed b51ab0d on 10.0.x
    Issue #3275237 by hooroomoo, lauriii, Wim Leers, nod_: Don't convert,...

  • lauriii committed e169a23 on 9.5.x
    Issue #3275237 by hooroomoo, lauriii, Wim Leers, nod_: Don't convert,...

  • lauriii committed eb82f6f on 9.4.x
    Issue #3275237 by hooroomoo, lauriii, Wim Leers, nod_: Don't convert,...
lauriii’s picture

Version: 10.0.x-dev » 9.3.x-dev

Committed b51ab0d and pushed to 10.0.x. Also committed #9 to 9.5.x and cherry-picked to 9.4.x. Thanks!

Leaving open for a 9.3.x commit once the freeze is over.

larowlan’s picture

Status: Reviewed & tested by the community » Fixed

Updating status

Wim Leers’s picture

Status: Fixed » Reviewed & tested by the community

Not yet backported 😅

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 9: 3275237-9-d93.patch, failed testing. View results

Wim Leers’s picture

Status: Needs work » Reviewed & tested by the community
lauriii’s picture

Status: Reviewed & tested by the community » Fixed

Committed f5aced4 and pushed to 9.3.x. Thanks!

  • lauriii committed f5aced4 on 9.3.x
    Issue #3275237 by hooroomoo, lauriii, Wim Leers, nod_: Don't convert,...

Status: Fixed » Closed (fixed)

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