Don't mangle the primary_blog option when trying to fix it. Don't set people to be a subscriber on a blog they already have a role for. See #15316
git-svn-id: http://svn.automattic.com/wordpress/trunk@16179 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
parent
2855be97ac
commit
037fae5ae6
|
@ -79,17 +79,17 @@ function get_active_blog_for_user( $user_id ) {
|
||||||
|
|
||||||
$primary_blog = get_user_meta( $user_id, 'primary_blog', true );
|
$primary_blog = get_user_meta( $user_id, 'primary_blog', true );
|
||||||
$first_blog = current($blogs);
|
$first_blog = current($blogs);
|
||||||
if ( $primary_blog ) {
|
if ( false !== $primary_blog ) {
|
||||||
if ( ! isset( $blogs[ $primary_blog ] ) ) {
|
if ( ! isset( $blogs[ $primary_blog ] ) ) {
|
||||||
add_user_to_blog( $first_blog->blog_id, $user_id, 'subscriber' );
|
update_user_meta( $user_id, 'primary_blog', $first_blog->userblog_id );
|
||||||
update_user_meta( $user_id, 'primary_blog', $first_blog->blog_id );
|
|
||||||
$primary = $first_blog;
|
$primary = $first_blog;
|
||||||
} else {
|
} else {
|
||||||
$primary = get_blog_details( $primary_blog );
|
$primary = get_blog_details( $primary_blog );
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
add_user_to_blog( $first_blog->blog_id, $user_id, 'subscriber' );
|
//TODO Review this call to add_user_to_blog too - to get here the user must have a role on this blog?
|
||||||
update_user_meta( $user_id, 'primary_blog', $first_blog->blog_id );
|
add_user_to_blog( $first_blog->userblog_id, $user_id, 'subscriber' );
|
||||||
|
update_user_meta( $user_id, 'primary_blog', $first_blog->userblog_id );
|
||||||
$primary = $first_blog;
|
$primary = $first_blog;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue