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
Unknown Object (File)
Sun, May 5, 1:33 PM
Unknown Object (File)
Fri, May 3, 8:18 AM
Unknown Object (File)
Sun, Apr 28, 9:13 PM
Unknown Object (File)
Sun, Apr 28, 12:00 PM
Unknown Object (File)
Thu, Apr 25, 2:16 AM
Unknown Object (File)
Wed, Apr 24, 6:37 AM
Unknown Object (File)
Fri, Apr 19, 6:02 PM
Unknown Object (File)
Fri, Apr 19, 4:15 PM
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