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

	F14034359: D7286.diff
	Sat, Nov 9, 11:48 PM

	F13999970: D7286.diff
	Thu, Oct 24, 6:22 PM

	Unknown Object (File)
	Oct 3 2024, 8:39 AM

	Unknown Object (File)
	Oct 3 2024, 8:36 AM

	Unknown Object (File)
	Oct 3 2024, 8:35 AM

	Unknown Object (File)
	Oct 3 2024, 8:34 AM

	Unknown Object (File)
	Sep 24 2024, 11:18 PM

	Unknown Object (File)
	Sep 20 2024, 8:37 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