Fix two issues where StandardPageView assumes it will always have access to a user
ClosedPublic
Actions

Authored by epriestley on Oct 11 2013, 2:40 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 20, 3:29 AM

	Unknown Object (File)
	Fri, Dec 13, 6:02 AM

	Unknown Object (File)
	Fri, Dec 13, 2:43 AM

	Unknown Object (File)
	Mon, Dec 9, 8:23 AM

	Unknown Object (File)
	Tue, Nov 26, 5:22 PM

	Unknown Object (File)
	Nov 18 2024, 3:38 AM

	Unknown Object (File)
	Nov 13 2024, 1:54 AM

	Unknown Object (File)
	Nov 9 2024, 11:48 PM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rP122f643d6f36: Fix two issues where StandardPageView assumes it will always have access to a…

Summary

If we hit an exception very early, the request may not have a user yet. Don't assume it will.

Test Plan

Broke MySQL config on purpose to test some stuff, loaded Phabricator, got a nice error page instead of a super-fatal one.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan accepted this revision.Oct 11 2013, 10:25 PM

Closed by commit rP122f643d6f36 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

view/

page/

PhabricatorStandardPageView.php

10 lines

Diff 16419

View Options

src/view/page/PhabricatorStandardPageView.php

Fix two issues where StandardPageView assumes it will always have access to a userClosedPublicActions