Changeset 4082

Timestamp:

09/16/2016 07:23:11 PM (8 years ago)

Author:

ocean90

Message:

Rosetta: Add locale managers as editors to the team site.

See #1983.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/rosetta/inc

Files:

• site/class-locale-main.php (modified) (1 diff)
• user/class-sync.php (modified) (3 diffs)

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

@@ -56 +56 @@
                 $user_sync = new User\Sync();
                 $user_sync->set_destination_site( $team_site );
-                $user_sync->set_roles_to_sync( [ 'editor' ] );
+                $user_sync->set_roles_to_sync( [
+                    'editor' => 'editor',
+                    Role\Locale_Manager::get_name() => 'editor',
+                ] );
                 $user_sync->setup();
             }

sites/trunk/wordpress.org/public_html/wp-content/plugins/rosetta/inc/user/class-sync.php

@@ -48 +48 @@
      * Sets the array of roles to sync.
      *
-     * @param array $roles_to_sync An array of roles to sync.
+     * @param array $roles_to_sync An array of roles to sync. The key of an item
+     *                             is the source role, the value the destination role.
      */
     public function set_roles_to_sync( array $roles_to_sync ) {
@@ -79 +80 @@
         remove_action( 'set_user_role',    [ $this, 'add_user_to_site' ], 10 );
 
-        if ( ! in_array( $role, $this->roles_to_sync, true ) ) {
+        if ( ! in_array( $role, array_keys( $this->roles_to_sync ), true ) ) {
             return;
         }
@@ -87 +88 @@
         }
 
-        add_user_to_blog( $this->destination_site->id, $user_id, $role );
+        add_user_to_blog( $this->destination_site->id, $user_id, $this->roles_to_sync[ $role ] );
     }
 }