Paths

Table of Contentst

Better 404 for Phame
ClosedPublic
Actions

Authored by chad on Jun 29 2016, 3:18 AM.

Tags

None

Referenced Files

	F19802671: D16192.id38949.diff
	Sun, Mar 1, 6:01 PM

	F19802669: D16192.id38949.diff
	Sun, Mar 1, 6:01 PM

	F19681338: D16192.id38949.diff
	Sun, Feb 8, 9:34 PM

	F19533107: D16192.id38949.diff
	Jan 20 2026, 3:33 PM

	F19533105: D16192.id38949.diff
	Jan 20 2026, 3:32 PM

	F19529310: D16192.id38949.diff
	Jan 19 2026, 3:36 PM

	F19319319: D16192.id38949.diff
	Dec 25 2025, 7:48 AM

	F19152041: D16192.diff
	Dec 11 2025, 12:19 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rP2b76785a13df: Better 404 for Phame

Summary

"Fixes" fatals on phacility.com blog. If post_id is either 0 or any other integer not in the blog system, show a normal Phame 404 with crumbs.

Test Plan

http://local.blog.phacility.com/post/0/last_published/, http://local.blog.phacility.com/post/999999/last_published/

Diff Detail

Repository

rP Phabricator

Branch

phame-500 (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 12827
Build 16341: Run Core Tests
Build 16340: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 38949.Jun 29 2016, 3:18 AM

chad retitled this revision from to Better 404 for Phame.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

Herald added a subscriber: Korvin. · View Herald TranscriptJun 29 2016, 3:18 AM

epriestley accepted this revision.Jun 29 2016, 4:33 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 29 2016, 4:33 AM

Closed by commit rP2b76785a13df: Better 404 for Phame (authored by chad, committed by chad). · Explain WhyJun 29 2016, 4:34 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

phame/

controller/

PhameLiveController.php

4 lines

Diff 38949

View Options

src/applications/phame/controller/PhameLiveController.php

Better 404 for PhameClosedPublicActions