Page MenuHomePhabricator
Differential D17163

Fix an issue which could prevent blocking reviewers from being removed from revisions
ClosedPublic
Actions

Authored by epriestley on Jan 9 2017, 4:20 PM.
Tags
None
Referenced Files
F15481016: D17163.id41272.diff
Tue, Apr 8, 5:46 PM
F15455333: D17163.id41271.diff
Sat, Mar 29, 11:34 PM
F15453799: D17163.id41272.diff
Sat, Mar 29, 2:26 PM
F15443294: D17163.id.diff
Thu, Mar 27, 3:33 AM
F15439377: D17163.diff
Wed, Mar 26, 7:16 AM
F15395255: D17163.id41271.diff
Mar 16 2025, 4:25 AM
F15299289: D17163.id41271.diff
Mar 5 2025, 2:39 PM
Unknown Object (File)
Feb 25 2025, 7:29 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T11114: Move Differential to EditEngine
Commits
rP425deeb52391: Fix an issue which could prevent blocking reviewers from being removed from…
Summary

Ref T11114. After evaluating typeahead tokens, we could process blocking reviewer removals incorrectly: we may get structures back.

Test Plan

Removed blocking reviewers from the web UI.

Diff Detail

Repository
rP Phabricator
Branch
literal1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15158
Build 19912: Run Core Tests
Build 19911: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
xaction/
8 lines

Diff 41271

View Options

src/applications/differential/xaction/DifferentialRevisionReviewersTransaction.php

Loading...
Phabricator · Built by Phacility