Last modified: 2013-07-01 23:42:22 UTC
In this call http://en.wikipedia.org/w/api.php?action=query&prop=imageinfo&titles=File%3APowderfinger-My_Happiness-23s.ogg&iiprop=size%7Curl&iiurlwidth=200 it returns thumbwidth="1913" thumbheight="-1" as a size of an thumb, which is completly wrong.
There are actually two bugs going on here. First, TimedMediaHandler isn't normalizing dimensions for audio files, which means that the height=-1 being passed to indicate "don't limit by height" is being misinterpreted. And second, TimedMediaHandler is using an incorrect calculation when the height is the limiting factor on the thumbnail dimensions, leading to the width of 1913. The same happens if you pass something like iiurlwidth=220&iiurlheight=10. Since both of these are in TimedMediaHandler, I'm going to reassign this there. I'll also submit a patch momentarily.
Related URL: https://gerrit.wikimedia.org/r/69869 (Gerrit Change Ide7208394f8495f5b143cbba215d4645598f1a50)
(In reply to comment #2) > Related URL: https://gerrit.wikimedia.org/r/69869 (Gerrit Change > Ide7208394f8495f5b143cbba215d4645598f1a50) This has been merged, and went out with wmf9. However, given that wmf8 is on the servers for an extra week and it makes really not-nice behaviour in VisualEditor, we probably want to back-port this.
Change 71393 had a related patch set uploaded by Catrope: Clean up transform dimension height handling https://gerrit.wikimedia.org/r/71393
Change 71393 merged by Catrope: Clean up transform dimension height handling https://gerrit.wikimedia.org/r/71393
This backport is now live; as such, marking this bug as fixed.
