Page MenuHomePhabricator

Fix some overlooked profile menu construction callsites
ClosedPublic

Authored by epriestley on Apr 9 2019, 9:24 PM.
Tags
None
Referenced Files
F22727753: D20385.id.diff
Sun, Oct 4, 4:07 AM
F22727739: D20385.id.diff
Sun, Oct 4, 4:06 AM
F22725008: D20385.id48646.diff
Sun, Oct 4, 1:55 AM
F22658973: D20385.id48647.diff
Thu, Oct 1, 7:17 AM
F22658970: D20385.id48646.diff
Thu, Oct 1, 7:17 AM
F22658969: D20385.id.diff
Thu, Oct 1, 7:17 AM
F22656336: D20385.diff
Thu, Oct 1, 3:55 AM
Unknown Object (File)
Thu, Sep 17, 4:23 AM
Subscribers
None

Details

Summary

Depends on D20384. Ref T13275. A bunch of this code got converted but I missed some callsites that aren't reached directly from the menu.

Test Plan
  • Visited each controller, saw actual pages instead of menu construction fatals.
  • Grepped for getProfileMenu().

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable