Page MenuHomePhabricator
Differential D19604

Replace "Disable User" web UI flow with transactions
ClosedPublic
Actions

Authored by epriestley on Aug 27 2018, 1:35 PM.
Tags
None
Referenced Files
F19510464: D19604.id46874.diff
Sun, Jan 11, 9:58 AM
F19307390: D19604.id46869.diff
Wed, Dec 24, 6:52 PM
F19104298: D19604.diff
Dec 5 2025, 1:17 PM
F18861046: D19604.diff
Nov 2 2025, 2:09 PM
F18816628: D19604.diff
Oct 21 2025, 8:18 AM
F18803953: D19604.id.diff
Oct 18 2025, 5:05 AM
F18718414: D19604.id.diff
Sep 29 2025, 3:56 PM
F18705874: D19604.diff
Sep 28 2025, 1:39 PM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13189: Plans: 2018 Week 35 Bonus Content
Commits
rP2f7b10c023b9: Replace "Disable User" web UI flow with transactions
Summary

Ref T13189. See PHI642. Upgrades the "Disable" action in the web UI to be transaction-based.

This technically breaks things a little (you can't disable non-bot users, since they now require CAN_EDIT and you won't have it) but an upcoming change will fix the permissions issue.

Test Plan

Disabled and enabled a (bot) user from the web UI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
src/
applications/
people/
controller/
13 lines

Diff 46874

View Options

src/applications/people/controller/PhabricatorPeopleDisableController.php

Loading...
Phabricator · Built by Phacility