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.
Version: master
Severity: normal
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | cscott | T56844 Image / media handling (tracking) | |||
| Duplicate | cscott | T51896 In Parsoid and MW API, implement handling for file types that do not have thumbnails, such as audio ogg files | |||
| Resolved | Anomie | T51921 imageinfo API returns incorrect data |
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