Deleting a Media view mode should not result in the deletion of an entire text format

Created on 11 December 2022, over 1 year ago

Updated 28 September 2023, 9 months ago

Problem/Motivation

Allowing the user to customize the media view modes that the user can select in the WYSIWYG unexpectedly creates a hard configuration dependency on the media view mode for the HTML format itself, even if the view mode is never used.

Steps to reproduce

Install Drupal 9.5.x HEAD with the Standard profile.
Enable the Media and Media Library modules.
On /admin/structure/display-modes/view/add/media/, create a new view mode for Media called "Test" and click save.
On /admin/structure/media/manage/image/display, under "Custom display settings", check "Test" and click save.
On /admin/config/content/formats/manage/basic_html, add the media button to the WYSIWYG toolbar.
Further down the page, check "Embed media".
Further down still, open the "Embed media" vertical tab that becomes available.
Under "View modes selectable in the 'Edit media' dialog", check "Default" and "Test". Click "Save".
Go to /admin/structure/display-modes/view/manage/media.test/delete. It warns you that not only the unused view mode but also the editor config and the basic HTML input format are about to be deleted:
If you don't read carefully, you may end up deleting the entire Basic HTML format, rendering your content both broken and uneditable in the process.

Proposed resolution

The selectable view modes for the media embed button should be treated as optional third-party settings that can be cleaned up from the editor and filter config entities, especially if they are never used. (Even if they are used, it'd be better to ignore a single data attribute rather than break the entire site's content.)

Remaining tasks

TBD

User interface changes

TBD

API changes

TBD

Data model changes

TBD

Release notes snippet

TBD

🐛 Bug report

Status

Needs work

Version

11.0 🔥

Component

Media →

Last updated 37 minutes ago

Maintained by
🇳🇱Netherlands @seanB
🇩🇪Germany @chr.fritsch
🇺🇸United States @phenaproxima
🇪🇸Spain @marcoscano

Created by

🇺🇸United States xjm

Live updates comments and jobs are added and updated live.

Needs followup

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment 9 months ago →
🇮🇳India AditiVB
i am working on this .
Issue was unassigned.
Status changed to Needs review 9 months ago11:06am 28 September 2023
Comment 9 months ago →
🇮🇳India AditiVB
When i keep the checkbox Allow the user to override the default view mode checked i am not getting the error .
Status changed to Needs work 9 months ago1:54pm 28 September 2023
Comment 9 months ago →
🇺🇸United States smustgrave
Following the steps of the IS can confirm the bug still on 11.x

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024