I agree that it needs to be adjusted in the search api as well.
It's in both, as the core calls a render hook that triggers the hook in the search api, preventing the render from being called unnecessarily is important.
This render in the webform is called even if it's not on a webform page or in the webform admin. All it takes is for some module to invoke the tokens that arrive in this function.
However, in the search api, I agree that there needs to be a rule to prevent unnecessary processing. I'm still investigating how to patch it.
Proposed solution
edmargomes → created an issue.
Fix once to use core/once library.
Example for work today:
let id = $('#edit-container-body-value').attr('data-ckeditor5-id');
Drupal.CKEditor5Instances.get(id).setData(email_content);
you can create a utils like this:
const ckeditorInstanceByElement = (elementId) => {
let id = $(elementId).attr('data-ckeditor5-id');
return Drupal.CKEditor5Instances.get(id);
};
I had a problem applying the .info patch for version 2.0.0
I'm sending the small adjustment here.
Fixed
edmargomes → created an issue.
ignore the problem is a custom patch applied after update the module.
edmargomes → created an issue.
Update the patch. I will continue the work
Started the process, I fixed problem with jquery once too.
Sending the first version to get more opinions
edmargomes → created an issue.
Fixed
edmargomes → created an issue.
The patch #5 works, but need change to get with one option, or config->storage->get or config->get
works for me
The same problem here with Drupal 9.5.3 and PHP 8.1
edmargomes → created an issue.
Works for me