Page MenuHomePhabricator

Rewrite email management to use AuthManager
Closed, ResolvedPublic

Description

  • Special:ChangeEmail

Related Objects

StatusSubtypeAssignedTask
Resolved Deskana
ResolvedAnomie
ResolvedAnomie
ResolvedNone
ResolvedTgr
ResolvedAnomie
OpenFeatureNone
ResolvedNone
ResolvedTgr
ResolvedTgr
ResolvedAnomie
ResolvedJoe
ResolvedJoe
Resolvedhashar
Resolvedbd808
ResolvedAnomie
ResolvedKrinkle
ResolvedNone
ResolvedJanZerebecki
ResolvedKrinkle
ResolvedTgr

Event Timeline

Tgr raised the priority of this task from to Needs Triage.
Tgr updated the task description. (Show Details)
Tgr subscribed.

This would also be a good time to refactor the email change business logic into a non-UI-related class and expose it as an API.

The part actually in SpecialChangeEmail was done as part of rMW3617c982c9db: Use AuthManager on special pages

However, it seems the code in User::setEmail has not significantly changed. Is this task done, or should that method be changed in some way involve AuthManager?

Tgr claimed this task.

I think I just forgot to close it. ChangeEmail was updated and PasswordReset was rewritten, and those were the only email management functions really related to authentication.