Changeset 1942 for sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles/rosetta-roles.php

Timestamp:

10/06/2015 03:05:07 PM (9 years ago)

Author:

ocean90

Message:

Rosetta: Allow to make users a Translation Editor even if they aren't a member of the site yet.

If a user isn't a member of the site add them as a subscriber.

Fixes #1237.

File:

• sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles/rosetta-roles.php (modified) (2 diffs)

sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles/rosetta-roles.php

@@ -315 +315 @@
 
                     if ( ! is_user_member_of_blog( $user_details->ID ) ) {
-                        wp_redirect( add_query_arg( array( 'error' => 'not-a-member' ), $redirect ) );
-                        exit;
+                        $added = add_existing_user_to_blog( array( 'user_id' => $user_details->ID, 'role' => 'subscriber' ) );
+                        if ( ! $added || is_wp_error( $added ) ) {
+                            wp_redirect( add_query_arg( array( 'error' => 'not-added-to-site' ), $redirect ) );
+                            exit;
+                        }
+
+                        // Refresh user data
+                        $user_details = get_user_by( 'id', $user_details->ID );
                     }
 
@@ -525 +531 @@
 
             'error' => array(
-                'no-user-found' => __( 'The user couldn’t be found.', 'rosetta' ),
-                'not-a-member'  => __( 'The user is not a member of this site.', 'rosetta' ),
-                'user-cannot'   => __( 'The user is not a translation editor.', 'rosetta' ),
-                'user-exists'   => __( 'The user is already a translation editor.', 'rosetta' ),
+                'no-user-found'     => __( 'The user couldn’t be found.', 'rosetta' ),
+                'not-a-member'      => __( 'The user is not a member of this site.', 'rosetta' ),
+                'not-added-to-site' => __( 'The user couldn’t be added to this site.', 'rosetta' ),
+                'user-cannot'       => __( 'The user is not a translation editor.', 'rosetta' ),
+                'user-exists'       => __( 'The user is already a translation editor.', 'rosetta' ),
             ),
         );