Add optional blogname to wp_get_sites

[elliott-stocks]

Currently to get the blogname for a list of sites it requires a call to get_blog_details() per site.

It'd be nice if we could add an optional parameter to wp_get_sites() which includes the blogname. I opted for a SQL query in the patch to prevent heavy use of switch_to_blog().

[elliott-stocks]

Code formatting fix

[jeremyfelt]

Hi @elliott-stocks, thanks for the report and the patch!

Expanding wp_get_sites() is definitely a goal. #31148 is working to allow searching by domain/path, which may effectively cover the blogname search for most scenarios.

Searching for data stored in individual options tables rather than in the global wp_site and wp_sitemeta tables inspires caution. I'm not sure we would want to go there yet.

[elliott-stocks]

Replying to jeremyfelt:

Hi @elliott-stocks, thanks for the report and the patch!

Expanding wp_get_sites() is definitely a goal. #31148 is working to allow searching by domain/path, which may effectively cover the blogname search for most scenarios.

Searching for data stored in individual options tables rather than in the global wp_site and wp_sitemeta tables inspires caution. I'm not sure we would want to go there yet.

Thanks for the feedback! I figured running the query would be quicker than calling get_blog_option() per site, as with a huge network that may result in a lot of unnecessary calls to switch_to_blog().

I'd be happy to hear your thoughts on improving this.

[jeremyfelt]

The best way forward for having blogname more dynamically accessible is probably via lazy loading meta when accessed for the first time on a WP_Site object. See #36935. That way we won't have to loop through get_blog_details() and we only pull the data when we need it.

wp_get_sites() itself should be deprecated (See #36994) in the near future, so I'm going to close this out as wontfix.