Page MenuHomePhabricator

Allow PhabricatorUserLog to store non-user PHIDs
ClosedPublic

Authored by epriestley on Jan 13 2014, 8:53 PM.

Details

Summary

Ref T4310. This is a small step toward separating out the session code so we can establish sessions for ExternalAccount and not just User.

Also fix an issue with strict MySQL and un-admin / un-disable from web UI.

Test Plan

Logged in, logged out, admined/de-admin'd user, added email address, checked user log for all those events.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Jan 13 2014, 8:57 PM
  • Fix some null issues for login failures, which are a slightly unusual case.
btrahan accepted this revision.Jan 14 2014, 12:04 AM
epriestley closed this revision.Jan 14 2014, 7:05 PM

Closed by commit rP220d680f3769 (authored by @epriestley).