Send message after successful mail change [#2016649]

Hello.
I needed this functionality: after successful user mail change, drupal must send message to old and new mail about this. Also I need tokens (old/new mail) in mail template.

Comment	File	Size	Author
#5	2016649-5-send_message_after_confirm.patch	7 KB	mErilainen
#5	7.x-1.x: PHP 5.5 & MySQL 5.5, D7 4 pass
#4	2016649-4-send-message-after-confirming.patch	7.02 KB	greggles
#4	7.x-1.x: PHP 5.5 & MySQL 5.5, D7 3 pass
#2	interdiff.txt	733 bytes	msgph
#2	email_confirm-send_message_after_change_mail-2016649-2.patch	7.8 KB	msgph
#2	7.x-1.x: PHP 5.5 & MySQL 5.5, D7 Patch failed to apply
#1	email_confirm-send_message_after_change_mail-2016649-1.patch	7.5 KB	VasilyKraev
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

File	Size
email_confirm-send_message_after_change_mail-2016649-1.patch	7.5 KB

Comment #2

msgph CreditAttribution: msgph at Wunder commented 13 October 2016 at 11:50

Issue summary:

View changes

File	Size
email_confirm-send_message_after_change_mail-2016649-2.patch	7.8 KB
7.x-1.x: PHP 5.5 & MySQL 5.5, D7 Patch failed to apply
interdiff.txt	733 bytes

Thanks for the patch, it works fine.

Here's an updated patch which includes the subject variable meta-information to hook_variable_info so it could be localized.

Comment #3

13 October 2016 at 12:53

Status:

Needs review

» Needs work

The last submitted patch, 2: email_confirm-send_message_after_change_mail-2016649-2.patch, failed testing.

Comment #4

greggles

he/him

English

Denver, Colorado, USA

CreditAttribution: greggles at CARD.com commented 13 October 2016 at 15:15

Status:

Needs work

» Needs review

File	Size
2016649-4-send-message-after-confirming.patch	7.02 KB
7.x-1.x: PHP 5.5 & MySQL 5.5, D7 3 pass

rerolling after removing the hunk that didn't apply. that hunk happened to be about making the drupal_set_message a 'warning' level even instead of 'ok' - and I think that kind of change should be in it's own issue instead of this one.

Comment #5

mErilainen CreditAttribution: mErilainen at Wunder commented 24 November 2016 at 10:14

Version:	7.x-1.1	» 7.x-1.x-dev
Status:	Needs review	» Reviewed & tested by the community

File	Size
2016649-5-send_message_after_confirm.patch	7 KB
7.x-1.x: PHP 5.5 & MySQL 5.5, D7 4 pass

Here is the patch refactored against the latest dev (works also for 1.3). I'm marking this RTBC because it doesn't include changes and I have tested it.