opendev/gerrit
Code Issues Proposed changes

Index changes after PostReviewers

Browse Source 
Change-Id: I36e71cbc309a761a68b6d9e7632faedb93714193
This commit is contained in:
Dave Borowitz 2013-09-13 15:01:23 -04:00
parent 8828458f53
commit 44a6a5a209
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gerrit-server/src/main/java/com/google/gerrit/server/change/PostReviewers.java
View File

@ -48,6 +48,7 @@ import com.google.gerrit.server.change.ReviewerJson.PostResult;
import com.google.gerrit.server.change.ReviewerJson.ReviewerInfo;
import com.google.gerrit.server.config.GerritServerConfig;
import com.google.gerrit.server.group.GroupsCollection;
import com.google.gerrit.server.index.ChangeIndexer;
import com.google.gerrit.server.mail.AddReviewerSender;
import com.google.gerrit.server.project.ChangeControl;
import com.google.gerrit.server.project.NoSuchProjectException;
@ -94,6 +95,7 @@ public class PostReviewers implements RestModifyView<ChangeResource, Input> {
private final ChangeHooks hooks;
private final AccountCache accountCache;
private final ReviewerJson json;
private final ChangeIndexer indexer;
@Inject
PostReviewers(AccountsCollection accounts,
@ -108,7 +110,8 @@ public class PostReviewers implements RestModifyView<ChangeResource, Input> {
@GerritServerConfig Config cfg,
ChangeHooks hooks,
AccountCache accountCache,
ReviewerJson json) {
ReviewerJson json,
ChangeIndexer indexer) {
this.accounts = accounts;
this.reviewerFactory = reviewerFactory;
this.addReviewerSenderFactory = addReviewerSenderFactory;
@ -122,6 +125,7 @@ public class PostReviewers implements RestModifyView<ChangeResource, Input> {
this.hooks = hooks;
this.accountCache = accountCache;
this.json = json;
this.indexer = indexer;
}
@Override
@ -255,6 +259,7 @@ public class PostReviewers implements RestModifyView<ChangeResource, Input> {
db.rollback();
}
indexer.index(rsrc.getChange());
accountLoaderFactory.create(true).fill(result.reviewers);
postAdd(rsrc.getChange(), result);
}