Display active theme name on Sites screen

[johnbillion]

A column should be added to the Sites screen in network admin to display the name of each site's active theme, possibly linked directly to wp-admin/themes.php.

[palmiak]

I'm not sure this should be in core.

Yet I've wrote code which does thing, but as I said - not sure shoud this be a plugin or core

<?php
// Hook to columns on sites.php
add_filter( 'wpmu_blogs_columns', 'theme_name_blog_column' );


function theme_name_blog_column( $sites_columns ) {
  // adding column
  $sites_columns = $sites_columns + [ 'theme_name' => __( 'Theme name' ) ];

  return $sites_columns;
}

add_action( 'manage_sites_custom_column', 'theme_name_custom_column', 10, 2 );


function theme_name_custom_column( $column_name, $blog_id ) {
    if ( $column_name == 'theme_name' ) {
        // if it's not default it should be stored as option
        $theme_name = get_blog_option( $blog_id, 'current_theme' );

        //if default used
        if ( ! $theme_name ) {
                $theme_name = wp_get_theme( WP_DEFAULT_THEME )->get( 'Name' );
        }

        $theme_url = get_admin_url( $blog_id, 'themes.php' );

        echo '<a href="'.$theme_url.'">'.$theme_name.'</a>';
    }
}

I'm not also sure where to put this code? In sites.php or somewhere else.

[palmiak]

I've read in #41167 that place to put stuff like would be ms-functions.php - If so, I can put this functions there.

[Bueltge]

If we implemented this inside the core, then should we also add a list, link in the theme view. Often admins view all themes and she should also listed which theme is in usage, in which site of the network. Maybe also, is a child active and if yes, which parent is necessary so that admins can maintain the list of themes.

I have implemented this in a plugin (info: ​https://github.com/bueltge/wordpress-multisite-enhancements), the feedback of the user get me the information about this view of content.

I think a new column in the site view is to short in this context.

[euthelup]

Hi,

In my opinion, this is a useful feature for admins but also for theme developers who gamble themes in multi-site installs on a daily basis (like me).

Because the multi-site context doesn't have a theme browser, I think that each theme in this column should link to the Appearance tab of the site, directly to the theme's modal.

Also, sites with child-themes should be flagged acordingly.

Should this column be sortable? I mean I'm not yet sure if it will help anyone, but it would be intuitive to order sites by the active theme? Anyway, this would require some patches to WP_Site_Query too since I don't think it supports an order by meta ATM.

In the same idea, a good question would be: When you click on a theme, it will make more sense to display all the sites with the selected theme? Just how the posts column works on the categories page.

[euthelup]

In the light of my last response(excepting the two questions), I come with this patch

[greatislander]

As discussed in ​Slack, this could take advantage of the blogmeta table to prevent excessive switch_to_blog() calls. It also might make sense to use switch_to_blog() once to collect several pieces of data for the Excerpt view. Some further discussion across a few tickets is probably needed. See also: #40268, #37923.

[flixos90]

@johnbillion In my opinion showing each site's active theme is a rather opinionated decision for core. I certainly agree that it can be helpful, but if we start showing as trivial things as the active theme in the sites list table, it would be hard to argue why not to show plugins there as well. All of that could easily make the table cluttered.

#41168 for that purpose makes a lot more sense to me.

The concerns about additional switch_to_blog() statements (as stated above), while not a blocker, make me more hesitant of this change.

I'd suggest closing this ticket, as this is plugin territory to me.