Just a very small patch to prevent notifications of profile updates done by user admin.

Perhaps this should be made into a config setting configurable in the admin.

CommentFileSizeAuthor
#7 advuser.module.patch1.7 KBAnonymous (not verified)
advuser.module.patch680 bytessyngi

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Closed (duplicate)

You posted the exact same patch at http://drupal.org/node/216994.

Anonymous’s picture

Anonymous (not verified) commented

I just noticed, it isn't exactly the same but use the original issue to update the original patch.

syngi’s picture

syngi commented

It's not the same issue

Anonymous’s picture

Anonymous (not verified) commented
Version: 5.x-1.x-dev » 5.x-2.x-dev
Status: Closed (duplicate) » Needs work

But it is the same set of lines in the original code in both patches. Combine the two works for the 5.x-2.x-dev version. You pick which issue to close. ;)

syngi’s picture

syngi commented

I always read "create separate patched for separate issues"?

Anonymous’s picture

Anonymous (not verified) commented

And if you look at the issue queues you will find it often requested of the patch creator to do this or that including merging on patch with another. If it involves the same line of code in the original then a subsequent patch might not match and cause extra work. We all like extra work. ;D

Anonymous’s picture

Anonymous (not verified) commented
Title: No user profile update notification when changed by user admin » No new user or user profile update notification when changed by user admin
Status: Needs work » Needs review
StatusFileSize
new advuser.module.patch1.7 KB

Here is a modified patch. I plan to put this into CVS soon. Please review and comment.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Fixed

Code has been committed.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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