Differential D17605

Don't downgrade accepts on update (fix "sticky accept")
ClosedPublic
Actions

Authored by epriestley on Apr 3 2017, 3:38 PM.

Tags

None

Referenced Files

	F19215247: D17605.diff
	Wed, Dec 17, 5:52 AM

	F19032667: D17605.id.diff
	Nov 25 2025, 7:13 AM

	F19025987: D17605.diff
	Nov 24 2025, 9:17 AM

	F18819846: D17605.id.diff
	Oct 22 2025, 10:13 AM

	F18817970: D17605.diff
	Oct 21 2025, 7:59 PM

	F18741077: D17605.id.diff
	Oct 2 2025, 8:27 AM

	F18714255: D17605.id42348.diff
	Sep 29 2025, 7:25 AM

	F18625942: D17605.id42348.diff
	Sep 15 2025, 11:58 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T12496: "differential.sticky-accept" isn't working because of overbroad accept downgrading

Commits

rP520d3b939204: (stable) Don't downgrade accepts on update (fix "sticky accept")
rP9ebb5f8cda56: Don't downgrade accepts on update (fix "sticky accept")

Summary

Fixes T12496. Sticky accept was accidentally impacted by the "void" changes in D17566.

Instead, don't always downgrade all accepts/rejects: on update, we only want to downgrade accepts.

Test Plan

With sticky accept off, updated an accepted revision: new state is "needs review".
With sticky accept on, updated an accepted revision: new state is "accepted" (sticky accept working correctly).
Did "reject" + "request review" to make sure that still works, worked fine.

Diff Detail

Repository

Branch

void1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 16275
Build 21629: Run Core Tests
Build 21628: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 3 2017, 3:38 PM

Harbormaster completed remote builds in B16275: Diff 42347.Apr 3 2017, 3:39 PM

chad accepted this revision.Apr 3 2017, 4:54 PM

This revision is now accepted and ready to land.Apr 3 2017, 4:54 PM

Closed by commit rP9ebb5f8cda56: Don't downgrade accepts on update (fix "sticky accept") (authored by epriestley, committed by epriestley). · Explain WhyApr 3 2017, 4:55 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

editor/

DifferentialTransactionEditor.php

14 lines

xaction/

DifferentialRevisionVoidTransaction.php

6 lines

Diff 42347

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...

src/applications/differential/xaction/DifferentialRevisionVoidTransaction.php

Loading...