Page MenuHomePhabricator
Differential D11702

People - refine permissions on creating new users
ClosedPublic
Actions

Authored by btrahan on Feb 6 2015, 12:45 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 23, 10:14 AM
Unknown Object (File)
Tue, Dec 17, 11:59 PM
Unknown Object (File)
Fri, Dec 13, 7:54 AM
Unknown Object (File)
Sat, Dec 7, 7:33 PM
Unknown Object (File)
Tue, Dec 3, 3:41 PM
Unknown Object (File)
Mon, Dec 2, 6:32 PM
Unknown Object (File)
Nov 26 2024, 4:28 PM
Unknown Object (File)
Nov 20 2024, 1:29 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T7142: Allow instances to create bot accounts
Commits
Restricted Diffusion Commit
rP345966cb4131: People - refine permissions on creating new users
Summary

Fixes T7142. Make old permission mean "make (non-bot) users" and then nuance the UI for those administrators who can make bot accounts.

Test Plan

loaded up admin a with full powers and admin b with restricted powers. noted admin a could make a full user. noted admin b could not make a full user. noted admin b got an error even via clever uri hacking.

Diff Detail

Repository
rP Phabricator
Branch
T7142
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4366
Build 4379: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 28158.Feb 6 2015, 12:45 AM
btrahan added a task: T7142: Allow instances to create bot accounts.
btrahan retitled this revision from to People - refine permissions on creating new users.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptFeb 6 2015, 12:45 AM
epriestley accepted this revision.Feb 6 2015, 12:46 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Feb 6 2015, 12:46 AM
Closed by commit rP345966cb4131: People - refine permissions on creating new users (authored by btrahan, committed by btrahan). · Explain WhyFeb 6 2015, 12:47 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
people/
application/
13 lines
capability/
2 lines
controller/
19 lines
4 lines

Diff 28158

View Options

src/applications/people/application/PhabricatorPeopleApplication.php

Loading...
View Options

src/applications/people/capability/PeopleCreateUsersCapability.php

Loading...
View Options

src/applications/people/controller/PhabricatorPeopleListController.php

Loading...
View Options

src/applications/people/controller/PhabricatorPeopleNewController.php

Loading...
Phabricator · Built by Phacility