Requesting a DEV that supports Drupal 10

Blocked by https://www.drupal.org/project/entity_embed/issues/3272732

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:

Comments

webengr created an issue. See original summary.

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

ranjit1032002’s picture

Status: Active » Needs review
StatusFileSize
new524 bytes

Created a patch for Drupal 10 compatibility, please review.
Thank You.

hardikpandya’s picture

StatusFileSize
new1.22 KB
new1.21 KB

With Drupal 10 here and reduced support for Drupal 8 going forward, I feel the info file should reflect the same. I have updated the core_version_requirement accordingly. Attached is the interdiff.

marcelovani’s picture

Thanks for the patches, but we are blocked by #3272732 https://www.drupal.org/project/entity_embed/issues/3272732

  • marcelovani committed 5bc1d3a8 on 8.x-1.x
    Issue #3348890 by Ranjit1032002, hardikpandya, webengr, marcelovani:...
marcelovani’s picture

Version: 2.0.x-dev » 2.x-dev
Status: Needs review » Needs work
marcelovani’s picture

Title: Drupal 10 compatibility? » Drupal 10 and Ckeditor5 compatibility
marcelovani’s picture

Title: Drupal 10 and Ckeditor5 compatibility » Drupal 10 and CKEditor5 compatibility
Issue summary: View changes
urvashi_vora’s picture

Status: Needs work » Needs review

For Drupal 10 and CKEditor5 compatibility:

We could try the following steps:
1. In the paragraphs_inline_entity_form.module file, add the following code:

use Drupal\editor\Entity\Editor;

function paragraphs_inline_entity_form_editor_settings_alter(array &$settings, Editor $editor) {
  if ($editor->getFilterFormat()->filters('filter_html')) {
    $settings['drupal']['paragraphs']['drupalSettings']['editor']['autoEmbed_media_external']['enabled'] = FALSE;
  }
}

2. In the composer.json file, add the following line to require the CKEditor5 package:

"drupal/ckeditor5": "1.0.x-dev",
"ckeditor/ckeditor": "^4.21"

3. Run composer update to install the CKEditor5 package.

4. In the paragraphs_inline_entity_form.libraries.yml file, add something like:

ckeditor5:
  remote: https://cdn.ckeditor.com/ckeditor5/29.0.0/classic/ckeditor.js
  minified: true
  ...
urvashi_vora’s picture

Status: Needs review » Needs work
marcelovani’s picture

Version: 2.x-dev » 8.x-1.x-dev
shivam_tiwari’s picture

Assigned: Unassigned » shivam_tiwari

shivam_tiwari’s picture

Assigned: shivam_tiwari » Unassigned
Status: Needs work » Needs review
marcelovani’s picture

@shivam_tiwari Thanks for the work on this, it will require some testing to validate the approach. Meanwhile, can you please rebase and update the pull request?

shivam_tiwari’s picture

@marcelovani I am not getting any changes here after rebase. Please check and let me know if anything.

marcelovani’s picture

StatusFileSize
new37.92 KB

If you go to https://git.drupalcode.org/project/paragraphs_inline_entity_form/-/merge... you will see the messages
Conflicts
Try to resolve on Gitlab itself by clicking the link on the message

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

darvanen’s picture

Status: Needs review » Needs work

Neither of these branches seem to contain a CKEditor 5 plugin. It install on Drupal 10 fine but CKEditor 5 compatibility is not ready yet.

nelo_drup’s picture

Module status for drupal 5

darvanen’s picture

Ok clearly I didn't know this system nearly well enough to be commenting in #21 about there being no plugin, that's further up the chain. I stand by my comment on the MR about the library definition though.

mortona2k’s picture

I didn't dig too deep, but I suspect the plugin will need a rewrite to be compatible with cke5.

darvanen’s picture

Status: Needs work » Needs review

This module does work with a cocktail of patches on its dependencies, but if you're hoping to insert entities in-text like our use-case you'll be blocked by the new `entity_embed` widget which only injects entities at the block level, between

elements, not inside them.

That said, as far this module is concerned, I'd say it passes manual testing (if you remove the 'ckeditor5' library definition from this module's library schema)

nelo_drup’s picture

Let's hope this module is updated because for now I can only use the following module https://www.drupal.org/project/paragraphs_entity_embed although it has its limitations

damienmckenna’s picture

Would it be worth splitting the ckeditor 5 integration from the Drupal 10 compatibility? If nothing else it'd make the pieces easier to test on their own.

damienmckenna’s picture

Also, if the changes for API compatibility mean that it's no longer compatible with DRupal 8, the info file should be updated to note this.

marcelovani’s picture

This project is very simple, it basically provides a custom Entity embed form that works as a glue to embed Paragraphs.

It depends on https://www.drupal.org/project/entity_embed/issues/3272732, that depends on https://www.drupal.org/project/embed/issues/3309747

Looks like these two issues are getting close to be resolved, but we are not there yet.

marcelovani’s picture

Looks like the dependencies have been fixed, I will try to allocate some time to check this.

marcelovani’s picture

Assigned: Unassigned » marcelovani
Status: Needs review » Needs work
marcelovani’s picture

lexfunk’s picture

Thank you everyone who has been contributing to this and all the related issues.

  1. https://www.drupal.org/project/entity_embed/issues/3396133 was fixed!
  2. I have installed the latest version of entity_embed
  3. I applied the patches from #32 WIP PR
  4. I enabled and configured CKEditor 5 for one of my text formats.
  5. This all seems to be working without any errors on Drupal 9.5.10, I will report back when I update to Drupal 10

  • marcelovani committed 856acbb9 on 3348890-drupal10-ckeditor5
    Issue #3348890: Updated dependencies.
    

  • marcelovani committed 77d9380d on 8.x-1.x
    Issue #3348890: Drupal 10 and CKEditor5 compatibility.
    
marcelovani’s picture

Thanks for checking @lexfunk and everyone that commented and tested this. I have just merged the PR https://git.drupalcode.org/project/paragraphs_inline_entity_form/-/merge...
I will make a new release shortly.

marcelovani’s picture

Assigned: marcelovani » Unassigned
Status: Needs work » Fixed

marcelovani’s picture

Uploaded a video on Youtube

Drupal 10 Demo Video

Drupal 10 Demo Video

Status: Fixed » Closed (fixed)

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

marcelovani’s picture

Status: Closed (fixed) » Needs work

marcelovani’s picture

I partially merged this PR in https://git.drupalcode.org/project/paragraphs_inline_entity_form/-/commi... as the PR was not mergeable.

marcelovani’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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