opendev/gerrit
Code Issues Proposed changes

Merge "Index change that is created on saving access rights for review"

Browse Source
This commit is contained in:
Edwin Kempin
2013-06-27 17:57:30 +00:00
committed by Gerrit Code Review
parent 122ecd55fc 1fbe95e445
commit 963206f375
1 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/project/ReviewProjectAccess.java
View File

@@ -32,6 +32,7 @@ import com.google.gerrit.server.change.PostReviewers;
import com.google.gerrit.server.git.GitRepositoryManager; import com.google.gerrit.server.git.GitRepositoryManager;
import com.google.gerrit.server.git.MetaDataUpdate; import com.google.gerrit.server.git.MetaDataUpdate;
import com.google.gerrit.server.git.ProjectConfig; import com.google.gerrit.server.git.ProjectConfig;
import com.google.gerrit.server.index.ChangeIndexer;
import com.google.gerrit.server.patch.PatchSetInfoFactory; import com.google.gerrit.server.patch.PatchSetInfoFactory;
import com.google.gerrit.server.project.ChangeControl; import com.google.gerrit.server.project.ChangeControl;
import com.google.gerrit.server.project.ProjectControl; import com.google.gerrit.server.project.ProjectControl;
@@ -63,17 +64,19 @@ public class ReviewProjectAccess extends ProjectAccessHandler<Change.Id> {
private final PatchSetInfoFactory patchSetInfoFactory; private final PatchSetInfoFactory patchSetInfoFactory;
private final Provider<PostReviewers> reviewersProvider; private final Provider<PostReviewers> reviewersProvider;
private final ChangeControl.GenericFactory changeFactory; private final ChangeControl.GenericFactory changeFactory;
private final ChangeIndexer indexer;
@Inject @Inject
ReviewProjectAccess(final ProjectControl.Factory projectControlFactory, ReviewProjectAccess(final ProjectControl.Factory projectControlFactory,
final GroupBackend groupBackend, GroupBackend groupBackend,
final MetaDataUpdate.User metaDataUpdateFactory, final ReviewDb db, MetaDataUpdate.User metaDataUpdateFactory, ReviewDb db,
final IdentifiedUser user, final PatchSetInfoFactory patchSetInfoFactory, IdentifiedUser user, PatchSetInfoFactory patchSetInfoFactory,
final Provider<PostReviewers> reviewersProvider, Provider<PostReviewers> reviewersProvider,
final ChangeControl.GenericFactory changeFactory, ChangeControl.GenericFactory changeFactory,
ChangeIndexer indexer,
@Assisted final Project.NameKey projectName, @Assisted Project.NameKey projectName,
@Nullable @Assisted final ObjectId base, @Nullable @Assisted ObjectId base,
@Assisted List<AccessSection> sectionList, @Assisted List<AccessSection> sectionList,
@Nullable @Assisted String message) { @Nullable @Assisted String message) {
super(projectControlFactory, groupBackend, metaDataUpdateFactory, super(projectControlFactory, groupBackend, metaDataUpdateFactory,
@@ -83,6 +86,7 @@ public class ReviewProjectAccess extends ProjectAccessHandler<Change.Id> {
this.patchSetInfoFactory = patchSetInfoFactory; this.patchSetInfoFactory = patchSetInfoFactory;
this.reviewersProvider = reviewersProvider; this.reviewersProvider = reviewersProvider;
this.changeFactory = changeFactory; this.changeFactory = changeFactory;
this.indexer = indexer;
} }
@Override @Override
@@ -122,6 +126,7 @@ public class ReviewProjectAccess extends ProjectAccessHandler<Change.Id> {
} finally { } finally {
db.rollback(); db.rollback();
} }
indexer.index(change);
return changeId; return changeId;
} }