WordPress Theme API Returning Broken Image

[downstairsdev]

If you search for "Portfolio Press" from the themes page of the WordPress dashboard it returns a broken image for my theme (​http://wordpress.org/themes/portfolio-press).

I think this has to do with the recent update we did to allow .jpg images to be used as the screenshot. My guess is that ​http://api.wordpress.org/themes/info/1.0/ automatically displays screenshot.png and doesn't check for screenshot.jpg.

[downstairsdev]

cc @otto42 by request.

[iandunn]

Yeah, it's trying to pull screenshot.png and getting a 404.

[Otto42]

Looks like we need to save the information about the screenshot name as meta at the time of the upload, then output the correct name from that later. I'll look into adding it.

[Otto42]

Theme directory should be showing JPG preferentially now, if the file exists. This determination is made at the time of the theme's approval, so if a screenshot.jpg exists then, it will use it, else it will fallback to the png file.

Portfolio Press was my test subject here, so it should be showing the JPG now. Still need to check the API response, but the themes directory is using the JPG at this point.

[downstairsdev]

Great! Thanks Otto.

[Otto42]

API response updated to reflect new screenshot info. Side effect, the URL is now a CDN hit instead of a direct one, and it should also be more SSL friendly.

I'm going to close this as fixed, for now. If other places where this happens are found, please re-open with info.