Page MenuHomePhabricator

VisualEditor: [Regression pre-wmf5] Make full size button remains disabled for any image in Betalabs
Closed, ResolvedPublic



Steps to reproduce:

1.Go to this page:
2.Select the image on the page
3.Go to Media Settings>Advanced Settings>Image Size
4.Change the tab from Default to custom or Custom to Default

Observed Result:

The "Make full size" button remains disabled for the image

See the screenshot attached

Version: unspecified
Severity: normal




Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:10 AM
bzimport set Reference to bz65314.
Ryasmeen created this task.May 14 2014, 8:59 PM

Found the steps to reproduce:

1.Insert any image
2.Go to Media Settings>Advanced Settings
3.Select Custom tab and change the values
4.Apply changes
5.Save the page
6.Reopen the page
7.Go to the Media Settings>Advanced Settings of that image again

Observed Result:
The "Make full size" button became disabled ,switching tabs/changing values again does not make it enabled anymore.

It seems likely that this is due to cache pollution of the promise cache introduced in . The bug only happens when saving then reopening the editor in the same session without navigating away.

Change 133384 had a related patch set uploaded by Catrope:
Revert "Cache image scalable promises by filename"

Change 133384 merged by jenkins-bot:
Revert "Cache image scalable promises by filename"

Verified the fix in Betalabs

Verified the fix in test2

Verified the fix in production