{"record":{"id":"6e35148ed5e115db","repo":"phacility/phabricator","slug":"you-can-not-request-review-of-this-revision-becaus-6e3514","errorCode":null,"errorMessage":"You can not request review of this revision because this revision is already under review and the action would have no effect.","messagePattern":"You can not request review of this revision because this revision is already under review and the action would have no effect\\.","errorType":"validation","errorClass":"Exception","httpStatus":null,"severity":"error","filePath":"src/applications/differential/xaction/DifferentialRevisionRequestReviewTransaction.php","lineNumber":102,"sourceCode":"\n  public function getActionName() {\n    return pht('Requested Review');\n  }\n\n  public function generateOldValue($object) {\n    return $object->isNeedsReview();\n  }\n\n  public function applyInternalEffects($object, $value) {\n    $status_review = DifferentialRevisionStatus::NEEDS_REVIEW;\n    $object\n      ->setModernRevisionStatus($status_review)\n      ->setShouldBroadcast(true);\n  }\n\n  protected function validateAction($object, PhabricatorUser $viewer) {\n    if ($object->isNeedsReview()) {\n      throw new Exception(\n        pht(\n          'You can not request review of this revision because this '.\n          'revision is already under review and the action would have '.\n          'no effect.'));\n    }\n\n    if ($object->isClosed()) {\n      throw new Exception(\n        pht(\n          'You can not request review of this revision because it has '.\n          'already been closed. You can only request review of open '.\n          'revisions.'));\n    }\n\n    $this->getActorSourceType($object, $viewer);\n  }\n\n  public function getTitle() {","sourceCodeStart":84,"sourceCodeEnd":120,"githubUrl":"https://github.com/phacility/phabricator/blob/5720a38cfe95b00ca4be5016dd0d2f3195f4fa04/src/applications/differential/xaction/DifferentialRevisionRequestReviewTransaction.php#L84-L120","documentation":"Error \"You can not request review of this revision because this revision is already under review and the action would have no effect.\" thrown in phacility/phabricator.","triggerScenarios":"Thrown at src/applications/differential/xaction/DifferentialRevisionRequestReviewTransaction.php:102 when the library encounters an invalid state.","commonSituations":"See trigger scenarios.","solutions":[],"exampleFix":null,"handlingStrategy":null,"validationCode":null,"typeGuard":null,"tryCatchPattern":null,"preventionTips":[],"tags":[],"backgroundTag":null,"analyzedSha":"5720a38cfe95b00ca4be5016dd0d2f3195f4fa04","analyzedAt":"2026-08-21T05:07:25.672Z","schemaVersion":2},"datasetVersion":"2026-08-21T11:28:35.574Z"}