From 04fecbc5aa52ec236afc6f5eec19bed08961f8dd Mon Sep 17 00:00:00 2001 From: Jeremy Felt Date: Wed, 29 Jun 2016 22:07:17 +0000 Subject: [PATCH] Multisite: Revert property type changes in `WP_Site`. Partial revert of [37657]. Moving existing properties in `WP_Site` from public to private broke backwards compatibility in a pretty severe way. We made an initial attempt to work around this, but due to the variety of possible issues, moving forward does not seem wise. Fixes #36717. git-svn-id: https://develop.svn.wordpress.org/trunk@37923 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/class-wp-site.php | 20 ++++---------------- 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/src/wp-includes/class-wp-site.php b/src/wp-includes/class-wp-site.php index d839c59600..5eb10a71c3 100644 --- a/src/wp-includes/class-wp-site.php +++ b/src/wp-includes/class-wp-site.php @@ -26,12 +26,10 @@ final class WP_Site { * A numeric string, for compatibility reasons. * * @since 4.5.0 - * @since 4.6.0 Converted from public to private to explicitly enable more intuitive - * access via magic methods. - * @access private + * @access public * @var string */ - private $blog_id; + public $blog_id; /** * Domain of the site. @@ -60,12 +58,10 @@ final class WP_Site { * A numeric string, for compatibility reasons. * * @since 4.5.0 - * @since 4.6.0 Converted from public to private to explicitly enable more intuitive - * access via magic methods. - * @access private + * @access public * @var string */ - private $site_id = '0'; + public $site_id = '0'; /** * The date on which the site was created or registered. @@ -234,10 +230,6 @@ final class WP_Site { switch ( $key ) { case 'id': return (int) $this->blog_id; - case 'blog_id': - return $this->blog_id; - case 'site_id': - return $this->site_id; case 'network_id': return (int) $this->site_id; case 'blogname': @@ -269,8 +261,6 @@ final class WP_Site { public function __isset( $key ) { switch ( $key ) { case 'id': - case 'blog_id': - case 'site_id': case 'network_id': return true; case 'blogname': @@ -300,10 +290,8 @@ final class WP_Site { public function __set( $key, $value ) { switch ( $key ) { case 'id': - case 'blog_id': $this->blog_id = (string) $value; break; - case 'site_id': case 'network_id': $this->site_id = (string) $value; break;