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
F13254556: D11702.diff
Sat, May 25, 3:30 AM
F13233881: D11702.diff
Tue, May 21, 2:38 AM
F13224742: D11702.id.diff
Sun, May 19, 10:33 AM
F13215444: D11702.id28158.diff
Fri, May 17, 5:23 PM
F13212159: D11702.diff
Fri, May 17, 6:26 AM
F13195741: D11702.diff
Sun, May 12, 10:35 PM
F13178444: D11702.diff
Wed, May 8, 8:25 PM
Unknown Object (File)
Apr 25 2024, 2:23 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 28159

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