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

	F13217775: D15274.id.diff
	Sat, May 18, 7:59 AM

	F13197505: D15274.diff
	Mon, May 13, 12:14 AM

	F13178780: D15274.diff
	Wed, May 8, 8:41 PM

	Unknown Object (File)
	Thu, Apr 25, 3:43 AM

	Unknown Object (File)
	Apr 12 2024, 2:18 AM

	Unknown Object (File)
	Apr 6 2024, 9:02 AM

	Unknown Object (File)
	Apr 6 2024, 5:17 AM

	Unknown Object (File)
	Apr 1 2024, 2:18 AM

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...