commit 6f334ea427c21101fce47ce1da2d666c65bac3d1
author Brad Larson <bklarson@gmail.com> Fri Feb 13 21:47:43 2009 -0600
committer Brad Larson <bklarson@gmail.com> Fri Feb 13 22:02:53 2009 -0600
tree 7e4dba95fc48a50ccdc5e314dc4f3e1098334278
parent 0b25c969e23d3ef23271fe83aca191eca4a1b3c1

Update approvals table when adding reviewer.

Fixed a bug where the Approvals table wasn't updating after adding a
reviewer, which made it difficult to see if the add actually worked.

Signed-off-by: Brad Larson <bklarson@gmail.com>

src/main/java/com/google/gerrit/client/changes/ApprovalTable.java

diff --git a/src/main/java/com/google/gerrit/client/changes/ApprovalTable.java b/src/main/java/com/google/gerrit/client/changes/ApprovalTable.java
index c2bb8f6..284e1a9 100644
--- a/src/main/java/com/google/gerrit/client/changes/ApprovalTable.java
+++ b/src/main/java/com/google/gerrit/client/changes/ApprovalTable.java
@@ -19,6 +19,7 @@
 import com.google.gerrit.client.data.AccountInfoCache;
 import com.google.gerrit.client.data.ApprovalDetail;
 import com.google.gerrit.client.data.ApprovalType;
+import com.google.gerrit.client.data.ChangeDetail;
 import com.google.gerrit.client.patches.PatchUtil;
 import com.google.gerrit.client.reviewdb.Account;
 import com.google.gerrit.client.reviewdb.ApprovalCategory;
@@ -27,6 +28,7 @@
 import com.google.gerrit.client.reviewdb.ChangeApproval;
 import com.google.gerrit.client.rpc.Common;
 import com.google.gerrit.client.rpc.GerritCallback;
+import com.google.gerrit.client.rpc.ScreenLoadCallback;
 import com.google.gerrit.client.ui.AccountDashboardLink;
 import com.google.gerrit.client.ui.AddMemberBox;
 import com.google.gwt.user.client.ui.ClickListener;
@@ -204,6 +206,16 @@
           public void onSuccess(final VoidResult result) {
             addMemberBox.setEnabled(true);
             addMemberBox.setText("");
+            Util.DETAIL_SVC.changeDetail(changeId,
+                new ScreenLoadCallback<ChangeDetail>() {
+                  public void onSuccess(final ChangeDetail r) {
+                    if (isAttached()) {
+                      setAccountInfoCache(r.getAccounts());
+                      display(r.getChange(), r.getMissingApprovals(), r
+                          .getApprovals());
+                    }
+                  }
+                });
           }
 
           @Override