Re-evaluate location of LoginUserMigrated hook
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Legoktm
	Apr 15 2015, 5:54 PM

Description

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

	Subject	Repo	Branch	Lines +/-
	Always pass a valid User object to LoginUserMigrated hook	mediawiki/core	master	+5 -2

Mentioned In: rMW4b504768553c: Always pass a valid User object to LoginUserMigrated hook
T96145: LoginUserMigrated hook causing Fatal Error: Argument 1 passed to CentralAuthUser::getInstance() must be an instance of User, bool given
Mentioned Here: T96145: LoginUserMigrated hook causing Fatal Error: Argument 1 passed to CentralAuthUser::getInstance() must be an instance of User, bool given

Legoktm raised the priority of this task from to Needs Triage.

Legoktm updated the task description. (Show Details)

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

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

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