phacility/phabricator · error · Exception

Reviewer "%s" is not a valid reviewer which you have authori

Error message

Reviewer "%s" is not a valid reviewer which you have authority to accept on behalf of.

What it means

Error "Reviewer "%s" is not a valid reviewer which you have authority to accept on behalf of." thrown in phacility/phabricator.

Source

Thrown at src/applications/differential/xaction/DifferentialRevisionAcceptTransaction.php:209

  }

  protected function validateOptionValue($object, $actor, array $value) {
    if (!$value) {
      throw new Exception(
        pht(
          'When accepting a revision, you must accept on behalf of at '.
          'least one reviewer.'));
    }

    // NOTE: We're including reviewers who have already been accepted in this
    // check. Legitimate users may race one another to accept on behalf of
    // packages. If we get a form submission which includes a reviewer which
    // someone has already accepted, that's fine. See T12757.

    list($options) = $this->getActionOptions($actor, $object, true);
    foreach ($value as $phid) {
      if (!isset($options[$phid])) {
        throw new Exception(
          pht(
            'Reviewer "%s" is not a valid reviewer which you have authority '.
            'to accept on behalf of.',
            $phid));
      }
    }
  }

  public function getTitle() {
    $new = $this->getNewValue();
    if (is_array($new) && $new) {
      return pht(
        '%s accepted this revision as %s reviewer(s): %s.',
        $this->renderAuthor(),
        phutil_count($new),
        $this->renderHandleList($new));
    } else {
      return pht(

View on GitHub (pinned to 5720a38cfe)

When it happens

Trigger: Thrown at src/applications/differential/xaction/DifferentialRevisionAcceptTransaction.php:209 when the library encounters an invalid state.

Common situations: See trigger scenarios.

AI-assisted analysis of phacility/phabricator@5720a38cfe (2026-08-21). Data as JSON: /api/errors/41f1d7e896528f88. Report an issue: GitHub.