Changeset 41380 for trunk/tests/phpunit/tests/multisite/network.php

Timestamp:

09/15/2017 11:14:49 AM (8 years ago)

Author:

flixos90

Message:

Multisite: Introduce get_main_site_id().

This function can be used to easily get the main site ID of a given network via the optional $network_id parameter, which defaults to the current network. The existing is_main_site() now uses the new function internally and now accepts an optional $network_id parameter as well.

The main purpose of the new function at this point is to ensure that the WP_Network::$blog_id property is always set. Magic getters in the class have been adjusted to auto-fill the property when it is accessed and empty. Furthermore the function encapsulates logic that was previously part of ms_load_current_site_and_network() and has been replaced with a call to the function now.

Props spacedmonkey, jeremyfelt, johnjamesjacoby, flixos90.
Fixes #29684.

File:

• trunk/tests/phpunit/tests/multisite/network.php (modified) (1 diff)

trunk/tests/phpunit/tests/multisite/network.php

@@ -584 +584 @@
         $this->assertEquals( $original_count + 1, $result );
     }
+
+    /**
+     * @ticket 29684
+     */
+    public function test_network_blog_id_set() {
+        $network = get_network( self::$different_network_id );
+
+        $this->assertSame( (string) self::$different_site_ids[0], $network->blog_id );
+    }
 }