Remove incorrect error message after reviewing a change

If the state has changed to one of CI states (STAGING, STAGED or INTEGRATING)
during review, error message is shown to the user. This incorrectly happened also
when commenting a change already in CI state. Now the error message is shown only
when review was initiated from non CI state but the state has changed to one of
CI states meanwhile, before saving review.

Task-number: QTQAINFRA-858
Change-Id: I3be7b56c6e3d80f9b4c3f8a0a7a97a7757da4ac3
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
Reviewed-by: Ismo Haataja <ismo.haataja@digia.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java
index 026a269e22..c8a7c57fb2 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java
@@ -86,6 +86,8 @@ public class PostReview implements RestModifyView<RevisionResource, Input> {
 
     /** Who to send email notifications to after review is stored. */
     public NotifyHandling notify = NotifyHandling.ALL;
+
+    public boolean changeReviewable;
   }
 
   public static enum DraftHandling {
@@ -179,7 +181,7 @@ public class PostReview implements RestModifyView<RevisionResource, Input> {
 
     Output output = new Output();
     output.labels = input.labels;
-    if (input.labels != null && change.getStatus().isCI()) {
+    if (input.labels != null && input.changeReviewable && change.getStatus().isCI()) {
       output.message =
           "The change was staged while you were reviewing it. " +
           "Due to this, only your comments were published, while your review scores were dropped.";