Adding users to a blog can set the primary_domain to an invalid value

[westi]

In add_user_to_blog we update the primary_domain of a user to the $blog_id passed in if the user doesn't have a primary_domain yet.

When adding an existing user to an existing site in a multisite setup we pass a blank string to this function .

In a shared usertable setup where the users were all defined elsewhere (e.g. shared with bbPress) this cause use to create blank primary_domain entries in the database.

This then causes the code in get_active_blog_for_user() to add the user as a subscriber on the current blog - even if actually they already had a role!

(In [16178]) When adding a user to the blog pass in the current blog id. See #15316.

Note: get_active_blog_for_user() is called by the admin bar which is why this has become really obvious.

(In [16179]) 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