Home GnuPG

Fix scope of "Accept" when you don't check all the "Force Accept" boxes
f1eeaaf59f7fUnpublished

Unpublished Commit ยท Learn More

Publishing Disabled: All publishing is disabled for this repository.

Description

Fix scope of "Accept" when you don't check all the "Force Accept" boxes

Summary:
Ref T12272. I wrote this correctly, then broke it by adding the simplification which treats "accept the defaults" as "accept everything".

This simplification lets us render "epriestley accepted this revision." instead of "epriestley accepted this revision onbehalf of: long, list, of, every, default, reviewer, they, have, authority, over." so it's a good thing, but make it only affect the reviewers it's supposed to affect.

Test Plan:

  • Did an accept with a force-accept available but unchecked.
  • Before patch: incorrectly accepted all possible reviewers.
  • After patch: accepted only checked reviewers.
  • Also checked the force-accept box, accepted, got a proper force-accept.

Reviewers: chad, lvital

Reviewed By: lvital

Maniphest Tasks: T12272

Differential Revision: https://secure.phabricator.com/D17634

Details

Provenance
epriestley <git@epriestley.com>Authored on Apr 6 2017, 11:56 PM
marcusPushed on Apr 11 2017, 6:41 PM
Parents
rPHABcefbdbcffe9e: Provide a "Reviewers" attachment to "differential.revision.search"
Branches
Unknown
Tags
Unknown

Event Timeline