Anope Bug Tracker - Anope Stable (2.0.x series)
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0001632||Anope Stable (2.0.x series)||[All Projects] General||public||2015-02-01 15:52||2015-03-12 14:16|
|Summary||0001632: Email change confirmation lists old email rather than new.|
|Description||The default data/example.conf has emailchange_message with "You have requested to change your email address to %e".|
Then in SendConfirmMail in modules/commands/ns_set.cpp, we replace %e with the account's current email.
This results in a message being sent to the user claiming "You have requested to change your email address to (OLD EMAIL)", when instead that should be the new email. (The email is, however, correctly sent to the new email)
One possible fix is to replace %e with the new_email var instead. However, some users of Anope may have already assumed %e is the old email and written their emailchange_message accordingly. I think for compatibility it would be wiser to leave %e as is, and introduce a new variable, perhaps named %E, that contains the new email address.
I attached a patch (exported with `git format-patch`, so should be ready to `git am`) to that effect, or open to discussion on the matter. I can submit a PR on github too if that is preferred.
|Steps To Reproduce||/ns set email firstname.lastname@example.org then check your email.|
|Tags||No tags attached.|
|Attached Files|| 0001-SendConfirmMail-Replace-E-with-new-email.patch (2,124) 2015-02-01 15:52|
|2015-02-01 15:52||ptseng||New Issue|
|2015-02-01 15:52||ptseng||File Added: 0001-SendConfirmMail-Replace-E-with-new-email.patch|
|2015-02-01 16:14||ptseng||Note Added: 0006694|
|2015-03-12 14:16||Adam||Note Added: 0006719|
|2015-03-12 14:16||Adam||Status||new => resolved|
|2015-03-12 14:16||Adam||Resolution||open => fixed|
|2015-03-12 14:16||Adam||Assigned To||=> Adam|