[wp-trac] [WordPress Trac] #38351: Replace get_blog_details() usage in wp-includes/ms-blogs.php with get_site()

[WordPress Trac]

#38351: Replace get_blog_details() usage in wp-includes/ms-blogs.php with
get_site()
--------------------------------+------------------------
 Reporter:  iamfriendly         |       Owner:
     Type:  enhancement         |      Status:  new
 Priority:  normal              |   Milestone:  4.7
Component:  Networks and Sites  |     Version:  4.4
 Severity:  normal              |  Resolution:
 Keywords:  has-patch           |     Focuses:  multisite
--------------------------------+------------------------
Changes (by flixos90):

 * keywords:   => has-patch
 * milestone:  Awaiting Review => 4.7


Comment:

 [attachment:38351.diff] replaces `get_blog_details()` calls in
 `refresh_blog_details()`, `update_blog_details()` and `get_blog_status()`.

 I didn't replace it for `get_blogaddress_by_id()` yet since the `$get_all`
 parameter is true there. I can see us do one of two things here:
 * either replace it anyway and rely on stating in the dev-note that the
 `site_details` filter must be used instead of the `blog_details` filter
 * or not replace it at this point, but do it at the same time we deprecate
 `get_blog_details()`

 What do you think?

--
Ticket URL: <https://core.trac.wordpress.org/ticket/38351#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform