Page MenuHomePhabricator
Differential D8591

Issue a proper 404 when trying to edit nonexistent events
ClosedPublic
Actions

Authored by epriestley on Mar 22 2014, 12:17 AM.
Tags
None
Referenced Files
F14759463: D8591.id.diff
Wed, Jan 22, 12:34 PM
Unknown Object (File)
Mon, Jan 13, 3:21 PM
Unknown Object (File)
Sat, Jan 11, 5:20 PM
Unknown Object (File)
Sat, Jan 11, 5:20 PM
Unknown Object (File)
Dec 5 2024, 1:45 PM
Unknown Object (File)
Nov 29 2024, 5:53 PM
Unknown Object (File)
Nov 18 2024, 8:55 PM
Unknown Object (File)
Oct 16 2024, 12:05 AM
View All Files
Subscribers
epriestley
Tokens
"Like" token, awarded by michiel3.

Details

Reviewers
btrahan
chad
Commits
Restricted Diffusion Commit
rPae03cb7fff69: Issue a proper 404 when trying to edit nonexistent events
Summary

We'll fatal a little later here when trying to call methods. 404 instead.

Test Plan

Visited /calendar/event/edit/9999999/ or similar.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20373.Mar 22 2014, 12:17 AM
epriestley retitled this revision from to Issue a proper 404 when trying to edit nonexistent events.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, chad.
Herald added a subscriber: epriestley.Mar 22 2014, 12:17 AM
chad accepted this revision.Mar 22 2014, 12:23 AM
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 22 2014, 12:23 AM
epriestley closed this revision.Mar 22 2014, 2:12 AM
epriestley updated this revision to Diff 20374.

Closed by commit rPae03cb7fff69 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
calendar/
controller/
3 lines

Diff 20374

View Options

src/applications/calendar/controller/PhabricatorCalendarEventEditController.php

Loading...
Phabricator · Built by Phacility