Merge Organic Group Membership & Admin status

If those two things don't include it, then this should also include group admin status. Probably in a way where if a user has two memberships in a group and one of those memberships is as an admin then that should be the one that is kept.

Are subscriptions really different or is that really about #640942: Merge notifications and messaging settings for a user?

Patch implements usermerge hooks on behalf of og for merging membership (and admin). It only updates the og_uid table, which is all I believe og uses.

Hmmn, the straight update could create problems. It needs to more intelligently merge.

We find the groups kept-user is not a member of and update them then we set blocked-user to inactive on any his or her's remaining groups.

This patch looks good to me. It handles the merge of users. It doesn't properly grant admin rights to "user to keep" if "user to delete" was an admin and "user to keep" isn't...but that seems like a small feature that may not be worth adding.

Leaving as "needs work" for that last point about elevating the "user to keep" to admin if the "user to delete" had admin rights.

I forgot to say the first part of that message:

This has been applied - http://drupal.org/cvs?commit=339220 - thanks, Ben!

I see the commit message in this thread, I followed the commit link and see D7 code, but I cannot find this code in 2.x or 2.x dev. Usermerge does not currently merge group membership (unless I'm missing something), and there is no mention in the code. Any idea why this code is lost?

This was committed to 6.x, but not ported to 7.x in [1203596] the commit that removes it is https://git.drupalcode.org/project/usermerge/commit/d84e1d08f8b96d671639...

So this is basically back to "Active" status as a feature request for 7.x.

Great, I may have a whack at it or get my company to sponsor the work. Thanks for the decade-later update!

So, a one line edit at Line 388 on usermerge.usermerge.inc:

             case 'other'  :

Will make group membership merge. I don't think this merges group roles, however, but as we are not using them currently it seems to work for us.

I'm worried there are other issues with doing this, but it's a starting point. Is there a way to pluck this field out of the "other" array and put it elsewhere?