Differential D11258

T6555, When creating a task with a non existing parent parameter, phabricator should 404
ClosedPublic
Actions

Authored by lpriestley on Jan 6 2015, 7:47 PM.

Tags

None

Referenced Files

	F15471093: D11258.diff
	Sat, Apr 5, 3:13 AM

	F15463999: D11258.id.diff
	Wed, Apr 2, 5:20 AM

	F15463477: D11258.id27022.diff
	Tue, Apr 1, 11:57 PM

	F15462605: D11258.id.diff
	Tue, Apr 1, 4:20 PM

	F15462383: D11258.id27023.diff
	Tue, Apr 1, 2:46 PM

	F15462329: D11258.id27022.diff
	Tue, Apr 1, 2:04 PM

	F15459071: D11258.diff
	Mon, Mar 31, 9:36 AM

	F15412835: D11258.diff
	Wed, Mar 19, 3:12 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T6555: When creating a task with a non existing parent parameter, there is no warning the parent task doesn't exist

Commits

Restricted Diffusion Commit
rP9c568802bb77: T6555, When creating a task with a non existing parent parameter, phabricator…

Summary

Fixes T6555, The following should 404: /maniphest/task/create/?parent=asdf, /maniphest/task/create/?parent=0, /maniphest/task/create/?parent=999999 (where T999999 does not exist)

Test Plan

Navigate to /maniphest/task/create/?parent=asdf or /maniphest/task/create/?parent=0 or /maniphest/task/create/?parent=999999 (where T999999 does not exist). See 404.

Diff Detail

Repository

Branch

subtaskghostparent

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3644
Build 3654: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 27022.Jan 6 2015, 7:47 PM

lpriestley retitled this revision from to T6555, When creating a task with a non existing parent parameter, phabricator should 404.

lpriestley updated this object.

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

lpriestley added a reviewer: epriestley.

lpriestley added a task: T6555: When creating a task with a non existing parent parameter, there is no warning the parent task doesn't exist.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 6 2015, 7:47 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.Jan 6 2015, 7:48 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jan 6 2015, 7:48 PM

Closed by commit rP9c568802bb77: T6555, When creating a task with a non existing parent parameter, phabricator… (authored by lpriestley, committed by epriestley). · Explain WhyJan 6 2015, 7:49 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

controller/

ManiphestTaskEditController.php

5 lines

Diff 27022

src/applications/maniphest/controller/ManiphestTaskEditController.php

Loading...