Page MenuHomePhabricator

Re-evaluate location of LoginUserMigrated hook
Closed, ResolvedPublic

Description

Currently it's possible for the hook to pass "false" in for $user, which caused T96145. That doesn't seem very useful.

Event Timeline

Legoktm created this task.Apr 15 2015, 5:54 PM
Legoktm raised the priority of this task from to Needs Triage.
Legoktm updated the task description. (Show Details)
Legoktm added subscribers: Legoktm, csteipp, bd808.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 15 2015, 5:54 PM

Change 211239 had a related patch set uploaded (by Legoktm):
Always pass a valid User object to LoginUserMigrated hook

https://gerrit.wikimedia.org/r/211239

Legoktm claimed this task.May 15 2015, 7:45 PM

Change 211239 merged by jenkins-bot:
Always pass a valid User object to LoginUserMigrated hook

https://gerrit.wikimedia.org/r/211239

Legoktm closed this task as Resolved.May 15 2015, 8:27 PM