Differential D15274

Improve subproject/milestone error handling for users who can't create projects
ClosedPublic
Actions

Authored by epriestley on Feb 15 2016, 3:22 PM.

Tags

None

Referenced Files

	F19508380: D15274.id36852.diff
	Sat, Jan 10, 3:47 AM

	F19498337: D15274.id.diff
	Thu, Jan 8, 12:58 PM

	F19343787: D15274.id.diff
	Fri, Dec 26, 12:21 AM

	F18906207: D15274.id36852.diff
	Nov 8 2025, 5:28 PM

	F18872574: D15274.diff
	Nov 5 2025, 5:10 AM

	F18850684: D15274.id.diff
	Oct 30 2025, 6:40 PM

	F18844285: D15274.diff
	Oct 29 2025, 4:25 AM

	F18843035: D15274.diff
	Oct 28 2025, 8:33 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10357: Adding milestones or subproject for non administrator show Unknown Object

Commits

Restricted Diffusion Commit
rPf1f8ee8e6ad5: Improve subproject/milestone error handling for users who can't create projects

Summary

Fixes T10357.

Show a better (more descriptive) error message when a user who can't create projects tries to create a subproject or milestone.
Disable the subproject actions if you don't have create permission.

All this stuff was already enforced properly: this diff doesn't make any actual policy changes, just improves the UI for users who lack permission.

Test Plan

As an unprivileged user (no "Can Create Projects"), tried to create a subproject or milestone.
After patch, got a disabled action, with more specific and helpful error than before.

Diff Detail

Repository

Branch

bperm1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 10696
Build 13144: Run Core Tests
Build 13143: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 36852.Feb 15 2016, 3:22 PM

epriestley retitled this revision from to Improve subproject/milestone error handling for users who can't create projects.

epriestley updated this object.

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

epriestley added a reviewer: chad.

epriestley added a task: T10357: Adding milestones or subproject for non administrator show Unknown Object.

epriestley mentioned this in T10357: Adding milestones or subproject for non administrator show Unknown Object.Feb 15 2016, 3:24 PM

chad accepted this revision.Feb 15 2016, 3:29 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Feb 15 2016, 3:29 PM

Closed by commit rPf1f8ee8e6ad5: Improve subproject/milestone error handling for users who can't create projects (authored by epriestley, committed by epriestley). · Explain WhyFeb 15 2016, 3:32 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

controller/

PhabricatorProjectEditController.php

5 lines

PhabricatorProjectSubprojectsController.php

7 lines

Diff 36852

src/applications/project/controller/PhabricatorProjectEditController.php

Loading...

src/applications/project/controller/PhabricatorProjectSubprojectsController.php

Loading...