Allow attribute mapping to be modified in MediaWysiwygFilter

Open on Drupal.org →

Created on 26 April 2025, 22 days ago

Problem/Motivation

It would be useful to be able to customize media mappings to allow other modules to modify the structures.

A good use case would be ckeditor_media_resize that can resize an image via ckeditor.

[[{"fid": "1146","view_mode": "default","fields": {"format": "default","field_file_image_alt_text[und][0][value]": false,"field_file_image_title_text[und][0][value]": false,},"type": "media","link_text": null,"field_deltas": {"1": {"format": "default","field_file_image_alt_text[und][0][value]": false,"field_file_image_title_text[und][0][value]": false,}},"attributes": {"height": 323,"width": 400,"style": "height: 283px; width: 350px; margin: 5px 8px; float: right;","class": "media-element file-default","data-delta": "1"}}]]

With ckeditor_media_resize this might be something like this:
<drupal-media data-entity-type="media" data-entity-uuid="00517131-5632-4716-a835-3e30f3200517" style="width:400px;" class="media-embed-resized">

Steps to reproduce

Proposed resolution

At the end of MediaWysiwygFilter::transform() initiate a hook that allows the attributes to be modified, e.g.:

hook_media_wysiwyg_filter_alter($embed_metadata, $source_attributes);
?>

Remaining tasks

Build the hook.

API changes

A new hook is available called hook_media_wysiwyg_filter_alter().

✨ Feature request

Status

Active

Version

1.0

Component

Code

Created by

🇺🇸United States DamienMcKenna NH, USA

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!17Allow attribute mapping to be modified in MediaWysiwygFilter
Open
🇺🇸United States DamienMcKenna
updated 22 days ago

Comments & Activities

Issue created by @DamienMcKenna
Merge request !17Allow MediaWysiwygFilter data to be altered. → (Open) created by DamienMcKenna
Comment 22 days ago →
🇺🇸United States DamienMcKenna NH, USA
A WIP and untested ;-) change to add hook_media_wysiwyg_filter_alter().
Pipeline finished with Failed
22 days ago
Total: 133s
#482695
Pipeline finished with Failed
22 days ago
Total: 147s
#482706
Pipeline finished with Failed
22 days ago
Total: 173s
#482779
Pipeline finished with Failed
22 days ago
Total: 146s
#482782
Comment 22 days ago →
🇺🇸United States DamienMcKenna NH, USA
FYI I've tested it now using code from the example hook implementation in the API docs, and it's working rather well for me, especially when used with 💬 Recommendation for converting alignments added via "style" attribute Active .
Comment 22 days ago →
🇺🇸United States DamienMcKenna NH, USA
I recreated the example hook implementation for CKEditor Media Resize, which was my intended use case: ✨ Convert data migrated using Media Migration to new structure Active
Comment 20 days ago →
🇺🇸United States DamienMcKenna NH, USA
I think the error message could be further refined to suggest creating the {file_managed} records and rerunning the migration.
Comment 20 days ago →
🇺🇸United States DamienMcKenna NH, USA

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024