flag_friend_user_cancel() needs to delete not just the entries for the deleted user's friends, but also all occurrences of the deleted user on others' friend lists.

CommentFileSizeAuthor
#2 patch_commit_69b57703f781.patch1.44 KBmoonray
#1 flag_friends-issue-1203666.patch742 bytesmoonray

Comments

moonray’s picture

moonray commented
Status: Active » Needs review
StatusFileSize
new flag_friends-issue-1203666.patch742 bytes

And here's a patch that'll fix the logic.

moonray’s picture

moonray commented
StatusFileSize
new patch_commit_69b57703f781.patch1.44 KB

Let's add an update function to delete existing flag_friend data from canceled accounts.

mikl’s picture

mikl
Location Møn
commented
Status: Needs review » Reviewed & tested by the community

Another good catch, should be merged.

sirkitree’s picture

sirkitree commented
Status: Reviewed & tested by the community » Fixed

Committed: http://drupalcode.org/project/flag_friend.git/commitdiff/8c5c031bee8ab44...

Thank you.

Status: Fixed » Closed (fixed)

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