diff options
author | Ismo Haataja <ismo.haataja@digia.com> | 2014-06-16 14:36:21 +0300 |
---|---|---|
committer | Ismo Haataja <ismo.haataja@digia.com> | 2014-06-19 12:30:58 +0200 |
commit | 0fd0dbe04ae8f7513d94a6ce83f5a8ea8e1e62d0 (patch) | |
tree | 564343a7389934fba90562fcc2876291b00a7fc9 /gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/PatchSetComplexDisclosurePanel.java | |
parent | 9f9d801a36e0b3de06f52be5215e8a4e02cfdd78 (diff) |
Always allow commenting a patch setv2.7.0-based
Prevent giving review scores but allow commenting for a change in
CI states (STAGING, STAGED or INTEGRATING). No need to block it with
error message like before. Also if the state has changed to one of
CI states while reviewing, review scores are dropped, just comments
are saved.
Task-number: QTQAINFRA-858
Change-Id: I684f004a1d680b2db184fedb50d936337f603721
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Reviewed-by: Ismo Haataja <ismo.haataja@digia.com>
Diffstat (limited to 'gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/PatchSetComplexDisclosurePanel.java')
-rw-r--r-- | gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/PatchSetComplexDisclosurePanel.java | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/PatchSetComplexDisclosurePanel.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/PatchSetComplexDisclosurePanel.java index ed519e9ead..ec737ea901 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/PatchSetComplexDisclosurePanel.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/PatchSetComplexDisclosurePanel.java @@ -634,15 +634,7 @@ class PatchSetComplexDisclosurePanel extends ComplexDisclosurePanel b.addClickHandler(new ClickHandler() { @Override public void onClick(final ClickEvent event) { - if (changeDetail.getChange().getStatus() == Change.Status.INTEGRATING - || changeDetail.getChange().getStatus() == Change.Status.STAGED - || changeDetail.getChange().getStatus() == Change.Status.STAGING) { - alertMessageBox(Util.C.headingReviewDisabled(), - Util.C.messageReviewDisabled()) - .center(); - } else { - Gerrit.display(Dispatcher.toPublish(patchSet.getId())); - } + Gerrit.display(Dispatcher.toPublish(patchSet.getId())); } }); actionsPanel.add(b); |