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

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.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/rosetta/inc/user/class-sync.php (modified) (3 diffs)

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 ] );
     }
 }