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

Timestamp:

09/16/2016 07:17:46 PM (8 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();
+            }
         }