Undefined array key "weight" when adding multiple media

Created on 6 April 2022, about 3 years ago

Updated 26 January 2024, about 1 year ago

Can't attach more than 1 media library item

Add multi-value media item (pdf file in my case). This field is set to be Unlimited
Save node
Edit node and try to add another media item.
Second media item not added.
Error when inspecting the page: Undefined array key \"weight\" in /app/web/core/modules/media_library/src/Plugin/Field/FieldWidget/MediaLibraryWidget.php on line 856

add validation if key weight is set

🐛 Bug report

Status

Needs work

Version

9.5

Component

Media →

Last updated about 4 hours ago

Created by

🇺🇸United States vlyalko

Live updates comments and jobs are added and updated live.

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

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 about 1 year ago →
🇵🇹Portugal joum Porto
Can confirm this still applies cleanly on Drupal 10.1.6.

Does it only require automated tests to move forward to RTBC?
First commit to issue fork.
Comment 3 months ago →
🇲🇽Mexico dalin 🇨🇦, 🇲🇽, 🌍
Here's my first attempt at an issue fork.
* Changed to "Null Coalescing Operator" because that's what previous patch was trying to do, but with way more complexity.
* Also fixed another similar warning in removeItem()

I was only experiencing PHP warnings, but the OP reported being completely unable to add multiple items on a media field that allows multiple items. In that case Drupal would be outright preventing you from what it's supposed to, so I'm upping to High priority.

Production build 0.71.5 2024