nb. https://fa.m.wikipedia.org/wiki/%DA%AF%D8%B1%D8%A8%D9%87#/media/File:Cat_poster_1.jpg?uselang=fa does not work, but https://fa.m.wikipedia.org/wiki/%DA%AF%D8%B1%D8%A8%D9%87#/media/File:Cat_poster_1.jpg does

Looks to fail around here, so tagging Thumbor

• mwversion: 1.41.0-wmf.30
• reqId: 9ab3256d-5f15-4483-9596-e06981716ce2
• Find reqId in Logstash

TransformationalImageHandler::doTransform: creating 800x525 thumbnail at mwstore://shared-multiwrite/local-thumb/0/0b/Cat_poster_1.jpg/800px-Cat_poster_1.jpg using scaler im

For https://fa.m.wikipedia.org/wiki/%DA%AF%D8%B1%D8%A8%D9%87#/media/File:Cat_poster_1.jpg?uselang=fa, ?uselang=fa is appended to the fragment of the URL, not the query string. MediaViewer then makes an imageinfo API query for File:Cat_poster_1.jpg?uselang=fa, which does not exist.

?uselang=fa is being added by https://fa.wikipedia.org/wiki/%D9%85%D8%AF%DB%8C%D8%A7%D9%88%DB%8C%DA%A9%DB%8C:Gadget-Direct-link-to-Commons.js . If I had to guess, this is a side effect of T127268: Dismantle ResourceLoader's "targets" system/T328610: Default gadgets to run on mobile as the script still works on desktop. The script is an old copy of https://www.mediawiki.org/wiki/Snippets/Direct_imagelinks_to_Commons with various fixes and changes piled on top. The current version of the script seems to have no effect on Minerva, but also does not include the uselang feature.

The simplest option is probably to only load this script on non-Minerva skins using |skins=vector,vector-2022,monobook,timeless,modern,cologneblue] in the gadget definition.

The simplest option is probably to only load this script on non-Minerva skins using |skins=vector,vector-2022,monobook,timeless,modern,cologneblue] in the gadget definition.

Seems like that would be prone to break in future and i'd be worried about it copied to over projects. Seems straightforward enough to fix the gadget so I'll do that later and update here.

After patching the code and on closer inspection this seems to be a bug in MobileFrontend.

When I run the following code, I see that the Thumbnail objects contain the query string in the file path.

mw.mobileFrontend.require('mobile.startup').currentPageHTMLParser().getThumbnails()

I suspect this code is at fault here.
https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/0652ba81c60c25d7714b7e965c251cd415caedda/src/mobile.startup/PageHTMLParser.js#L173C2-L173C2

Patch very much welcome.