WordPress.org
Get WordPress

Make WordPress Core

Ticket #36717: 36717.10.diff

File 36717.10.diff, 3.8 KB (added by jeremyfelt, 8 years ago)

  • src/wp-includes/class-wp-network.php

     
    1818 *
    1919 * @since 4.4.0
    2020 *
    21  * @property int $id
    2221 * @property int $site_id
    2322 */
    2423class WP_Network {
    … …  
    2726         * Network ID.
    2827         *
    2928         * @since 4.4.0
    30          * @since 4.6.0 Converted from public to private to explicitly enable more intuitive
    31          *              access via magic methods. As part of the access change, the type was
    32          *              also changed from `string` to `int`.
    33          * @access private
     29         * @access public
    3430         * @var int
    3531         */
    36         private $id;
     32        public $id;
    3733
    3834        /**
    3935         * Domain of the network.
    … …  
    6561         * @access private
    6662         * @var string
    6763         */
    68         private $blog_id = '0';
     64        public $blog_id = '0';
    6965
    7066        /**
    7167         * Domain used to set cookies for this network.
    … …  
    154150         */
    155151        public function __get( $key ) {
    156152                switch ( $key ) {
    157                         case 'id';
    158                                 return (int) $this->id;
    159                         case 'blog_id':
    160                                 return $this->blog_id;
    161153                        case 'site_id':
    162154                                return (int) $this->blog_id;
    163155                }
    … …  
    178170         */
    179171        public function __isset( $key ) {
    180172                switch ( $key ) {
    181                         case 'id':
    182                         case 'blog_id':
    183173                        case 'site_id':
    184174                                return true;
    185175                }
    … …  
    200190         */
    201191        public function __set( $key, $value ) {
    202192                switch ( $key ) {
    203                         case 'id':
    204                                 $this->id = (int) $value;
    205                                 break;
    206                         case 'blog_id':
    207193                        case 'site_id':
    208194                                $this->blog_id = (string) $value;
    209195                                break;

  • src/wp-includes/class-wp-site.php

     
    2626         * A numeric string, for compatibility reasons.
    2727         *
    2828         * @since 4.5.0
    29          * @since 4.6.0 Converted from public to private to explicitly enable more intuitive
    30          *              access via magic methods.
    31          * @access private
     29         * @access public
    3230         * @var string
    3331         */
    34         private $blog_id;
     32        public $blog_id;
    3533
    3634        /**
    3735         * Domain of the site.
    … …  
    6058         * A numeric string, for compatibility reasons.
    6159         *
    6260         * @since 4.5.0
    63          * @since 4.6.0 Converted from public to private to explicitly enable more intuitive
    64          *              access via magic methods.
    65          * @access private
     61         * @access public
    6662         * @var string
    6763         */
    68         private $site_id = '0';
     64        public $site_id = '0';
    6965
    7066        /**
    7167         * The date on which the site was created or registered.
    … …  
    234230                switch ( $key ) {
    235231                        case 'id':
    236232                                return (int) $this->blog_id;
    237                         case 'blog_id':
    238                                 return $this->blog_id;
    239                         case 'site_id':
    240                                 return $this->site_id;
    241233                        case 'network_id':
    242234                                return (int) $this->site_id;
    243235                        case 'blogname':
    … …  
    269261        public function __isset( $key ) {
    270262                switch ( $key ) {
    271263                        case 'id':
    272                         case 'blog_id':
    273                         case 'site_id':
    274264                        case 'network_id':
    275265                                return true;
    276266                        case 'blogname':
    … …  
    300290        public function __set( $key, $value ) {
    301291                switch ( $key ) {
    302292                        case 'id':
    303                         case 'blog_id':
    304293                                $this->blog_id = (string) $value;
    305294                                break;
    306                         case 'site_id':
    307295                        case 'network_id':
    308296                                $this->site_id = (string) $value;
    309297                                break;

  • tests/phpunit/tests/multisite/network.php

     
    8787                return 3;
    8888        }
    8989
    90         /**
    91          * @ticket 37050
    92          */
    93         function test_wp_network_object_id_property_is_int() {
     90        function test_wp_network_object_id_property_is_string() {
    9491                $id = self::factory()->network->create();
    9592
    9693                $network = WP_Network::get_instance( $id );
    9794
    98                 $this->assertSame( (int) $id, $network->id );
     95                $this->assertSame( (string) $id, $network->id );
    9996        }
    10097
    10198        /**