Page MenuHomePhabricator

Diffusion - return 404 errors for bad URIs
ClosedPublic

Authored by btrahan on Jan 9 2015, 9:06 PM.
Tags
None
Referenced Files
F13237117: D11299.id27153.diff
Tue, May 21, 12:27 PM
F13237116: D11299.id27151.diff
Tue, May 21, 12:27 PM
F13237115: D11299.id27150.diff
Tue, May 21, 12:27 PM
F13237058: D11299.id.diff
Tue, May 21, 12:18 PM
F13236808: D11299.diff
Tue, May 21, 11:04 AM
F13219522: D11299.id27150.diff
Sat, May 18, 7:11 PM
F13204636: D11299.diff
Wed, May 15, 12:47 AM
F13189380: D11299.diff
Sat, May 11, 5:54 AM
Subscribers

Details

Summary

Fixes T5646. Makes diffusion a much better user experience. Users now see a 404 exception page when they have a bad URI. Previously, they saw a developer-facing raw exception.

Test Plan

played around in diffusion a bunch. most of these changes were fairly mechanical at the end of the day.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Diffusion - return 404 errors for bad URIs.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
src/applications/diffusion/controller/DiffusionChangeController.php
11

i'll tighten this up

src/applications/diffusion/controller/DiffusionCommitBranchesController.php
9–12

this does nothing...

src/applications/diffusion/controller/DiffusionCommitController.php
14–25

this is a problem; will fix

epriestley edited edge metadata.

You caught everything I caught. Looks good modulo your one inline.

This revision is now accepted and ready to land.Jan 9 2015, 9:18 PM
btrahan edited edge metadata.
  • updates
  • also fixed DiffController, where I forgot to initialize $data originally
This revision was automatically updated to reflect the committed changes.