Add users from Admin section always default role?

I had to think about this…

When you add users to the site, what page do they go to when they complete registration?

Are these NEW users to the network or existing users?

Also what options do you have on the JMM plugin? That is, are you auto-adding users to sites (when a logged in user visits, they have a role assigned) or what?

I have a theory, but I need to know what your JMM settings are before I can reproduce it fully 🙂

Ah, that would make sense then. The new users visit the site and get the default role, because that gets called via the automatic membership.

If the users came to the site to log in AND confirm their account on the subsite, that may be getting triggered before/at the same time as the user is added to the site… I wonder if you could trip the same filter with http://wordpress.org/plugins/multisite-user-management/

I’m going to have to play with this to sort out the order of things. I mean, I’m already checking if( !is_user_member_of_blog() ) so if that isn’t set yet, then this totally makes sense. JMM is running before WP sets the user to be a member of the site (or they may be running at the same time). Weird…

It happens because for SOME reason WP isn’t actaully adding the user to the site.

So … here.

1) You add a new user

2) User comes to the site to activate account

3) JMM adds the user as it’s default user role

4) WP sees the user has a role and ignores what you picked in #1

The thing is I don’t know why #4 happens last. It should be done at step 2 (the user comes to the site and activates account which gives it a role), but apparently WP adding the new user to the site is being triggered later than I’d thought.

I’m going to have to walk through the code backwards and see why the creation+registration is not occurring at the same time with WP. I may have to add in a check to see if we’re registering a new user, instead of JUST checking if the user is a member of the site. Dunno how to do that yet!

I don’t think that’s the problem, since it’s doing this:

add_user_to_blog($blog_id, $current_user->ID, $jmm_options['role']);

If it was tiggering when YOU added the user, then Current User is you, and that would have already bailed on the check for !is_user_member_of_blog() (which you are… )

Wait… is YOUR id the member of the site?

And is this happening right away (like you add the user and BOOM they’re default-role) before they confirm the account?

ETA: Also of note, join_site isn’t called automatically. It’s triggered via do_action on specific events.

You did it here? https://plugins.trac.wordpress.org/browser/join-my-multisite/trunk/lib/shortcode.php#L24

So it became this?

/* The registration magic */
	function jmm_activate_user( $user_id, $password, $meta ) {
	    global $blog_id;
            $role = (isset($_POST['role']) && isset($_POST['user_login'])) ? $_POST['role'] : get_site_option( 'default_user_role', 'subscriber' );
	    add_user_to_blog( $blog_id, $user_id, $role );
	}
	add_action( 'wpmu_activate_user', 'jmm_activate_user', 10, 3 );

Why get_site_option( 'default_user_role', 'subscriber' ); and not
get_option( 'default_role' );

That obviates the default role setting in the plugin.

/* The registration magic */
	function jmm_activate_user( $user_id, $password, $meta ) {
	    global $blog_id;
            if ( isset($_POST['role']) && isset($_POST['user_login']) ) {
                $role = $_POST['role'];
            } else {
                get_option( 'default_role' );
            }
	    add_user_to_blog( $blog_id, $user_id, $role );
	}
	add_action( 'wpmu_activate_user', 'jmm_activate_user', 10, 3 );

(I prefer to break it out for readability when debugging later 😉 )

Can you test my version that I posted? If that works, I’ll bang on it a little more for some stress testing, and then get in some sanitizing on the post.