Problem/Motivation

Ok, this was a fun one to track down. See #2548713-81: Only one additional new value saved unlimited field and no non-field values are restored after preview for the exact regression that the issue there caused, and we managed to hit this rarest of all rare cases ;)

Easy to work around by pressing Upload first

We also have some *really* weird stuff in there with setting hidden width/height which I figured out is because of some low-level typed data validation. This can easily be avoided if we use real image data, because the problem we work around there is that the code fails to detect width/height of the image. Cleaned that up as well.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 paragraphs-access-test-fails-2836242-2.patch2.16 KBBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Active » Needs review
FileSize
paragraphs-access-test-fails-2836242-2.patch2.16 KB
johnchque’s picture

johnchque
Primary language Spanish
Location Bolivia
CreditAttribution: johnchque at MD Systems GmbH commented
Status: Needs review » Reviewed & tested by the community

Hmm so that's why we needed the width and height. Cool! :)

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Committed, thx!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.