Prepopulate image properties when editing an embedded image

Can verify this is a huge issue.

+1

This might be a core issue. The same behavior takes place using the image dialog without IMCE/uploads, just providing a path to an image. Switching back/fourth to source within CKEditor without saving also causes the bug.

This is related to the injected image tag's "data-entity-type" and "data-entity-uuid" attributes being empty. The plugin doesn't bind to these elements.

Example, an image that works, uploaded using core modules.

<img alt="test" data-caption="Hey waddup earph" data-entity-type="file" data-entity-uuid="c70cca1e-15c6-4e4b-8fb1-ac28c0498d85" src="/sites/default/files/small_bear.jpg" />

Example, an image provided by IMCE as a path, does not work.

<img alt="test" data-caption="Hey waddup earph" data-entity-type="" data-entity-uuid="" src="/sites/default/files/small_bear.jpg" />

Attached is an attempted core patch for this. It just ensures these properties are filled in (not empty).

It may be better to patch this wherever the image manipulation plugin binds. This way the plugin is tolerant of empty type/uuid attributes, which seems like a proper expression of the data. This approach provides values to trick the system into processing the image, which might cause other issues.

A similar problem and with a closer solution in: #2850277: IMCE image caption support

Thanks!

@medinasod, I believe that the issue solved this problem correctly without the use of a false uuid

@medinasod, there a problem security on the patch of the issue #2850277: IMCE image caption support

+++ b/core/modules/ckeditor/js/plugins/drupalimage/plugin.js
@@ -90,6 +90,21 @@
+            if(typeof element.attributes['data-entity-uuid'] == 'undefined'){
+              element.attributes['data-entity-uuid'] = '0';
+            }

@see, it does not generate uuid, only add 0

@medinasod, can you test the last patch in the #2850277: IMCE image caption support?

Fixed on #2850277: IMCE image caption support!

Did you uninstall and reinstall IMCE?

@medinasod, would are this?

Thanks!