The notifications/messaging settings - http://drupal.org/project/notifications http://drupal.org/project/messaging - and any current messages about to be sent should also be merged to the new user.

Comments

sjtout’s picture

On a similar note, I added this line in order to merge Subscriptions (http://drupal.org/project/subscriptions):

 db_query('UPDATE {subscriptions} SET recipient_uid = %d where recipient_uid = %d', $user_to_keep->uid, $user_to_delete->uid);

I knew my site would need it so just hardcoded it in, but you could probably test for the 'subscriptions' table and only include that line if the table exists -- or something like that?

This module has been very helpful in adding LDAP accounts & authentication to a site that was already working for a while and had Drupal only accounts. Thank you!

greggles’s picture

Priority: Normal » Critical

It's also necessary to delete the notifications_queue entries for the deleted user.

coltrane’s picture

Status: Active » Needs review
StatusFileSize
new2.51 KB
greggles’s picture

I think this either has to be smarter or dumber ;)

They could be duplicates on some other field like the send_interval or the send_method. So, let's just delete the subscriptions for the user_to_delete.

greggles’s picture

Status: Needs review » Needs work
coltrane’s picture

Status: Needs work » Needs review
StatusFileSize
new1.88 KB

Dumbed.

greggles’s picture

Status: Needs review » Fixed

Also fixed - http://drupal.org/cvs?commit=339222 - thanks, Ben!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 523b595 on master, 8.x-1.x, 7.x-2.x, 7.x-2.x-api by greggles:
    feature #640942 by coltrane | greggles: Merge notifications and...