diff options
Diffstat (limited to 'gerrit-gwtui/src/main/java/com/google/gerrit/client/change/QuickApprove.java')
-rw-r--r-- | gerrit-gwtui/src/main/java/com/google/gerrit/client/change/QuickApprove.java | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/QuickApprove.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/QuickApprove.java index c4a74f545a..56cc7a7852 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/QuickApprove.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/change/QuickApprove.java @@ -24,6 +24,7 @@ import com.google.gerrit.client.rpc.GerritCallback; import com.google.gerrit.client.rpc.Natives; import com.google.gerrit.common.PageLinks; import com.google.gerrit.reviewdb.client.Change; +import com.google.gerrit.reviewdb.client.Project; import com.google.gwt.core.client.JsArrayString; import com.google.gwt.event.dom.client.ClickEvent; import com.google.gwt.event.dom.client.ClickHandler; @@ -33,6 +34,7 @@ import com.google.gwtexpui.safehtml.client.SafeHtmlBuilder; /** Applies a label with one mouse click. */ class QuickApprove extends Button implements ClickHandler { private Change.Id changeId; + private Project.NameKey project; private String revision; private ReviewInput input; private ReplyAction replyAction; @@ -47,7 +49,7 @@ class QuickApprove extends Button implements ClickHandler { setVisible(false); return; } - if (info.revision(commit).isEdit() || info.revision(commit).draft()) { + if (info.revision(commit).isEdit()) { setVisible(false); return; } @@ -71,6 +73,7 @@ class QuickApprove extends Button implements ClickHandler { if (qName != null) { changeId = info.legacyId(); + project = info.projectNameKey(); revision = commit; input = ReviewInput.create(); input.drafts(DraftHandling.PUBLISH_ALL_REVISIONS); @@ -93,14 +96,14 @@ class QuickApprove extends Button implements ClickHandler { if (replyAction != null && replyAction.isVisible()) { replyAction.quickApprove(input); } else { - ChangeApi.revision(changeId.get(), revision) + ChangeApi.revision(project.get(), changeId.get(), revision) .view("review") .post( input, new GerritCallback<ReviewInput>() { @Override public void onSuccess(ReviewInput result) { - Gerrit.display(PageLinks.toChange(changeId)); + Gerrit.display(PageLinks.toChange(project, changeId)); } }); } |