Changeset 41861 for trunk/tests/phpunit/tests/multisite/getMainSiteId.php

Timestamp:

10/14/2017 10:41:15 PM (7 years ago)

Author:

flixos90

Message:

Multisite: Take WP_Network::$blog_id into account in get_main_site_id().

When the WP_Network::$blog_id property is set manually, for example in the multisite bootstrap process, get_main_site_id() should use that value instead of running its own logic. The main logic for the function was therefore moved into the internal WP_Network::get_main_site_id() method, which is now being accessed by the function through the magic property handling for WP_Network::$blog_id (and its equivalent WP_Network::$site_id).

Props spacedmonkey, jeremyfelt.
Fixes #41936.

File:

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

trunk/tests/phpunit/tests/multisite/getMainSiteId.php

@@ -106 +106 @@
         return 333;
     }
+
+    /**
+     * @ticket 41936
+     */
+    public function test_get_main_site_id_with_property_value() {
+        global $current_site;
+
+        $original_main_site_id = $current_site->blog_id;
+        $current_site->blog_id = '123';
+
+        $result = get_main_site_id();
+
+        $current_site->blog_id = $original_main_site_id;
+
+        $this->assertSame( 123, $result );
+    }
+
+    /**
+     * @ticket 41936
+     */
+    public function test_get_main_site_id_filtered_with_property_value() {
+        global $current_site;
+
+        $original_main_site_id = $current_site->blog_id;
+        $current_site->blog_id = '123';
+
+        add_filter( 'pre_get_main_site_id', array( $this, 'filter_get_main_site_id' ) );
+        $result = get_main_site_id();
+
+        $current_site->blog_id = $original_main_site_id;
+
+        $this->assertSame( 333, $result );
+    }
 }