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
F14069929: D11702.diff
Wed, Nov 20, 10:36 AM
F14055654: D11702.diff
Sat, Nov 16, 2:24 PM
F14043424: D11702.diff
Tue, Nov 12, 10:06 AM
F14025591: D11702.diff
Thu, Nov 7, 6:56 PM
F13973391: D11702.diff
Oct 18 2024, 12:43 AM
F13967281: D11702.id.diff
Oct 16 2024, 12:43 PM
Unknown Object (File)
Oct 9 2024, 1:11 PM
Unknown Object (File)
Oct 2 2024, 7:36 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