Patch to add email support to buddylist.module [#77154]

This patch allows email to be sent to users when they are added to, or removed from, someone's buddylist. This email is configurable on the buddylist configurations page and can contain a series of replacement tags such as %adder_name, %addee_name, %site, etc...

Good stuff!

-jeff

Comment	File	Size	Author
	buddylist-mail.patch	6.7 KB	jjeff

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

ajwwong CreditAttribution: ajwwong commented 7 September 2006 at 01:48

This seems to work quite well...

Excellent!! Thanks so much...

The one thing that I couldn't get to work was filling in the "alternate text" in the form. I'm not sure if I applied the patch right, but for some reason, even though all the variables were appropriately set, e.g., buddylist_add_message and buddylist_remove_message [in the final buddy_mail_user function], I still couldn't get those messages to substitute in instead of the hard-coded defaults.

Thanks, though for a big step forward for buddylist!

Comment #2

robertDouglass CreditAttribution: robertDouglass commented 7 September 2006 at 09:02

Thanks for testing. Please confirm that you tested against the latest 4.7.0 branch and not CVS. Any chance that you could look into the issue with the replacement variables and reroll?

Comment #3

ajwwong CreditAttribution: ajwwong commented 7 September 2006 at 16:44

Hey Robert,

Just fyi.... I'm testing on a heavily modded-out version of my own "friends by mutual consent / invite-only" style buddylist module... and therefore I did this test as hand applied (and modified for my own needs patch) .... so my test above is definitely not a "controlled testing environment" :-)

But maybe someone who's working with a fresh copy of buddylist can run with it... 'cause this is definitely a *great* feature addition for buddylist.

I'm kinda swimming in my own stuff just trying to keep up with the some of the new traffic on my site and make sure it's up and running. But I'll definitely try to look into this more when I can.

Take care,
Albert