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
F18193390: D8591.id20374.diff
Sun, Aug 17, 5:20 AM
F18107132: D8591.diff
Sun, Aug 10, 11:42 PM
F18057268: D8591.id.diff
Mon, Aug 4, 8:57 AM
F18056948: D8591.id20374.diff
Mon, Aug 4, 8:51 AM
F18053298: D8591.id20373.diff
Mon, Aug 4, 4:27 AM
F18050447: D8591.id20373.diff
Sun, Aug 3, 7:48 PM
F18036093: D8591.diff
Sun, Aug 3, 1:47 AM
F17870237: D8591.id.diff
Jul 28 2025, 1:32 PM
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