Changeset 4081

Timestamp:

09/16/2016 07:17:46 PM (10 years ago)

Author:

ocean90

Message:

Rosetta: Don't use get_site_by_path() for fetching the team site.

get_site_by_path() returns the main site if the team site doesn't exist yet because it also searches sites with path = '/'.

See #1983.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/rosetta/inc/site/class-locale-main.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/rosetta/inc/site/class-locale-main.php

@@ -45 +45 @@
         public function register_events() {
                 if ( is_admin() ) {
-                        $user_sync = new User\Sync();
-                        $user_sync->set_destination_site( get_site_by_path( get_site()->domain, Locale_Team::$path ) );
-                        $user_sync->set_roles_to_sync( [ 'editor' ] );
-                        $user_sync->setup();
+                        // Get the team site.
+                        $result = get_sites( [
+                                'domain' => get_site()->domain,
+                                'path'   => Locale_Team::$path,
+                                'number' => 1,
+                        ] );
+                        $team_site = array_shift( $result );
+
+                        if ( $team_site ) {
+                                $user_sync = new User\Sync();
+                                $user_sync->set_destination_site( $team_site );
+                                $user_sync->set_roles_to_sync( [ 'editor' ] );
+                                $user_sync->setup();
+                        }
                 }