Simplify form logic in media library

Open on Drupal.org →

Created on 9 February 2022, over 3 years ago

Updated 19 January 2023, over 2 years ago

Problem/Motivation

After digging deeper into the form api, I found out that it might be possible to simplify some form handling.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

📌 Task

Status

Needs review

Version

10.1 ✨

Component

Last updated 1 day ago

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

Created by

🇩🇪Germany chr.fritsch 🇩🇪🇪🇺🌍

Live updates comments and jobs are added and updated live.

Needs issue summary update
Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Incomplete comments

Sign in to follow issues

Merge Requests

!1793Simplify form logic in media library
Closed
🇩🇪Germany chr.fritsch
updated 2 months ago
!6511Simplify form logic in media library
Closed
🇫🇷France mably
updated 2 months ago
!10843Simplify form logic in media library
Open
🇫🇷France DrDam
updated 3 months ago

Comments & Activities

Not all content is available!

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

Merge request !1793Resolve #3263397 "Simplify form logic" → (Closed) created by chr.fritsch
Status changed to Needs work over 2 years ago11:05pm 13 February 2023
Comment over 2 years ago →
🇺🇸United States smustgrave
For the issue summary requested in #4
Comment over 1 year ago →
🇫🇷France mably
Patch #16 doesn't apply to Drupal 9.5.11.
Merge request !6511Issue #3263397 by @mably: Simplify form logic in media library (Drupal 9.5.11 compatible version) → (Closed) created by mably
First commit to issue fork.
Merge request !10843reroll Patch #16 on 11.x → (Open) created by DrDam
Comment 5 months ago →
🇫🇷France DrDam
Create MR for 11.x and reroll #16 on it
Pipeline finished with Failed
5 months ago
#389544
First commit to issue fork.
Pipeline finished with Success
4 months ago
Total: 748s
#415218
Pipeline finished with Failed
4 months ago
#415228
Pipeline finished with Failed
4 months ago
Total: 544s
#415235
First commit to issue fork.
Pipeline finished with Success
3 months ago
Total: 649s
#443358
Comment 3 months ago →
🇬🇧United Kingdom the_g_bomb
Comment 3 months ago →
🇬🇧United Kingdom the_g_bomb
I hope the addition to the description satisfies the request in #4
Pipeline finished with Success
3 months ago
Total: 523s
#444839
Comment 2 months ago →
System Message
smustgrave → closed merge request !6511
Status changed to Needs review 2 months ago5:58pm 24 March 2025
Comment 2 months ago →
System Message
smustgrave → closed merge request !1793
Comment about 1 month ago →
🇺🇸United States smustgrave
Fact some tests had to be updated to pass makes me think could be a breaking change for some
Status changed to Needs work 5 days ago6:34pm 26 May 2025
Comment 5 days ago →
🇺🇸United States smustgrave
What if someone extending the media library doesn't have these new keys what are the odds it'll be an immediate breaking.

Also not sure '_weight' is standard key format? But moving to NW as there appear to be open questions from #4

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024