Page MenuHomePhabricator

Fix a bad getURI() call in Profile Panel handle construction
ClosedPublic

Authored by epriestley on Jul 7 2016, 9:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 18, 11:11 PM
Unknown Object (File)
Feb 12 2024, 7:05 AM
Unknown Object (File)
Feb 8 2024, 8:00 PM
Unknown Object (File)
Feb 3 2024, 11:19 AM
Unknown Object (File)
Jan 17 2024, 7:50 PM
Unknown Object (File)
Dec 27 2023, 8:26 AM
Unknown Object (File)
Dec 21 2023, 3:38 PM
Unknown Object (File)
Dec 21 2023, 1:10 PM
Subscribers
None

Details

Summary

Fixes T11285. We can end up loading panel handles while processing edits (e.g., disabling menu items on a project). We probably started loading these after the modular transaction changes in T9789, which load the handle for the transaction object unconditionally.

The handles aren't too useful, but they currently fail to load/build because panels don't have a URI. We could give them some sort of method here, but just nuke it for now since they don't appear anywhere and this unclogs the daemon queue.

Test Plan
  • Disabled a menu item on a project.
  • Ran publish task with bin/worker execute --id <id>.
  • Before patch: fatal on getURI() with stack trace similar to T11285.
  • After patch: clean execution.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fix a bad getURI() call in Profile Panel handle construction.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, avivey.
avivey edited edge metadata.
This revision is now accepted and ready to land.Jul 7 2016, 9:55 PM
This revision was automatically updated to reflect the committed changes.