Page MenuHomePhabricator
Differential D16192

Better 404 for Phame
ClosedPublic
Actions

Authored by chad on Jun 29 2016, 3:18 AM.
Tags
None
Referenced Files
F15387696: D16192.id38949.diff
Sat, Mar 15, 2:47 AM
F15340757: D16192.diff
Sun, Mar 9, 4:39 PM
F15335573: D16192.diff
Sat, Mar 8, 4:32 PM
Unknown Object (File)
Fri, Feb 28, 2:11 PM
Unknown Object (File)
Jan 30 2025, 4:30 AM
Unknown Object (File)
Jan 27 2025, 7:14 PM
Unknown Object (File)
Jan 27 2025, 7:01 PM
Unknown Object (File)
Jan 22 2025, 7:55 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

PathSize
src/
applications/
phame/
controller/
4 lines

Diff 38949

View Options

src/applications/phame/controller/PhameLiveController.php

Loading...
Phabricator · Built by Phacility