Opened 9 years ago
Closed 9 years ago
#38351 closed enhancement (fixed)
Replace get_blog_details() usage in wp-includes/ms-blogs.php with get_site()
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.7 | Priority: | normal |
| Severity: | normal | Version: | 4.4 |
| Component: | Networks and Sites | Keywords: | has-patch |
| Focuses: | multisite | Cc: |
Description
Attachments (2)
Change History (9)
#2
@
9 years ago
- Keywords has-patch added
- Milestone changed from Awaiting Review to 4.7
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_detailsfilter must be used instead of theblog_detailsfilter - or not replace it at this point, but do it at the same time we deprecate
get_blog_details()
What do you think?
#4
@
9 years ago
38351.2.diff includes the additional replacement.
I personally think we're fine replacing get_blog_details() regardless of the parameters and document it the filter change. We could theoretically also add an apply_filters_deprecated( 'blog_details', array( $details ), '4.7.0', 'site_details' ) into the relevant part of WP_Site to give developers time to migrate.
get_blogaddress_by_id(),refresh_blog_details(),update_blog_details(),get_blog_status()numbers 13-16 from https://core.trac.wordpress.org/ticket/37102#comment:17