WordPress doesn't report the currently active theme

[josephscott]

In rev [11714] WordPress stopped reporting the currently active theme to api.wordpress.org - see ticket #10220 for more history.

I've created a small patch to fix this, while I was in there I renamed the field 'active_theme' instead of 'template' to be more descriptive.

[ryan]

Should it use 'stylesheet' instead of 'template'?

[westi]

The whole naming convention in this function is a little sucky it seems.

I guess we should clear some of that up too.

[westi]

$current_theme->active_stylesheet = get_option( 'stylesheet' );
Should probably add so we get parent and child theme

[ryan]

Does api.wordpress.org need any fields other than Name and Version. It seems we are sending a lot of useless data.

[ryan]

Also, current_theme is being filled with the previous update response and then stuck into the request. Seems current_theme when sent in the request should not have checked and response fields.

[ryan]

That cleans things up a bit. $current_theme is gone. Instead the current theme slug is assigned to the 'current_theme' key of the themes array. This gets rid of all of the local cache junk that was being sent to api under the current_theme key. The themes array is populated with only the Name and Version for each theme. That's all I see being used api side. The last cached update and the new update are in better named $last_update and $new_update variables.

[ryan]

Send more theme fields to api. Blacklist unneeded fields.

[ryan]

It seems we need to send more than just version, name, and slug to help with disambiguation. Updated patch to blacklist certain fields and let the rest through.

[josephscott]

I don't think we need Description, Tags or Screenshot fields either.

[ryan]

(In [15455]) Clean up wp_update_themes(). Fix reporting of active theme. Props josephscott. fixes #14381 for 3.1

[ryan]

(In [15456]) Clean up wp_update_themes(). Fix reporting of active theme. Props josephscott. fixes #14381 for 3.0.1