Differential D17566

Fix an issue where "Request Review" of a fully-accepted revision would transition to "Accepted"
ClosedPublic
Actions

Authored by epriestley on Mar 28 2017, 12:52 PM.

Tags

None

Referenced Files

	F18474941: D17566.id.diff
	Tue, Sep 2, 11:01 PM

	F18459058: D17566.diff
	Mon, Sep 1, 4:10 PM

	F18291107: D17566.id42263.diff
	Sat, Aug 23, 10:51 AM

	F18230063: D17566.id42270.diff
	Wed, Aug 20, 11:56 AM

	F18230061: D17566.id42263.diff
	Wed, Aug 20, 11:56 AM

	F18230058: D17566.id.diff
	Wed, Aug 20, 11:56 AM

	F18220450: D17566.diff
	Tue, Aug 19, 2:47 PM

	F18087131: D17566.id42263.diff
	Aug 6 2025, 2:33 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10967: Move Differential reviewers back to a dedicated database storage table

Commits

rPaea46e55dac0: Fix an issue where "Request Review" of a fully-accepted revision would…

Summary

Ref T10967. This is explained in more detail in T10967#217125

When an author does "Request Review" on an accepted revision, void (in the sense of "cancel out", like a bank check) any "accepted" reviewers on the current diff.

Test Plan

Create a revision with author A and reviewer B.
Accept as B.
"Request Review" as A.
(With sticky accepts enabled.)
Before patch: revision swithced back to "accepted".
After patch: the earlier review is "voided" by te "Request Review", and the revision switches to "Review Requested".

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 28 2017, 12:52 PM

Harbormaster completed remote builds in B16188: Diff 42263.Mar 28 2017, 12:53 PM

epriestley mentioned this in D17568: When voiding "Accept" reviews, also void "Reject" reviews.Mar 28 2017, 1:24 PM

chad accepted this revision.Mar 28 2017, 5:28 PM

This revision is now accepted and ready to land.Mar 28 2017, 5:28 PM

Closed by commit rPaea46e55dac0: Fix an issue where "Request Review" of a fully-accepted revision would… (authored by epriestley, committed by epriestley). · Explain WhyMar 28 2017, 6:50 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in rPddc02ce420a5: When voiding "Accept" reviews, also void "Reject" reviews.

epriestley mentioned this in D17605: Don't downgrade accepts on update (fix "sticky accept").Apr 3 2017, 3:38 PM

epriestley mentioned this in rP9ebb5f8cda56: Don't downgrade accepts on update (fix "sticky accept").Apr 3 2017, 4:55 PM

epriestley mentioned this in rP520d3b939204: (stable) Don't downgrade accepts on update (fix "sticky accept").Apr 5 2017, 1:35 AM

epriestley mentioned this in T10828: File deletion should be queued up and run by daemons.Apr 9 2017, 11:13 AM

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20170328.reviewers.01.void.sql

2 lines

src/

__phutil_library_map__.php

2 lines

applications/

differential/

editor/

DifferentialTransactionEditor.php

13 lines

storage/

DifferentialReviewer.php

9 lines

xaction/

DifferentialRevisionReviewTransaction.php

41 lines

DifferentialRevisionVoidTransaction.php

63 lines

Diff 42270

resources/sql/autopatches/20170328.reviewers.01.void.sql

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...

src/applications/differential/storage/DifferentialReviewer.php

Loading...

src/applications/differential/xaction/DifferentialRevisionReviewTransaction.php

Loading...

src/applications/differential/xaction/DifferentialRevisionVoidTransaction.php

Loading...