diff --git a/java/com/google/gerrit/server/restapi/change/RevertSubmission.java b/java/com/google/gerrit/server/restapi/change/RevertSubmission.java index 47c6a7e226..9d2bd037af 100644 --- a/java/com/google/gerrit/server/restapi/change/RevertSubmission.java +++ b/java/com/google/gerrit/server/restapi/change/RevertSubmission.java @@ -95,11 +95,12 @@ public class RevertSubmission String.format("change is %s.", ChangeUtil.status(changeResource.getChange()))); } - String submissionId = - requireNonNull( - changeResource.getChange().getSubmissionId(), - String.format("merged change %s has no submission ID", changeResource.getId())); - + String submissionId = changeResource.getChange().getSubmissionId(); + if (submissionId == null) { + throw new ResourceConflictException( + "This change is merged but doesn't have a submission id," + + " meaning it was not submitted through Gerrit."); + } List changeDatas = queryProvider.get().bySubmissionId(submissionId); for (ChangeData changeData : changeDatas) {