| // Copyright (C) 2015 The Android Open Source Project | |
| // | |
| // Licensed under the Apache License, Version 2.0 (the "License"); | |
| // you may not use this file except in compliance with the License. | |
| // You may obtain a copy of the License at | |
| // | |
| // http://www.apache.org/licenses/LICENSE-2.0 | |
| // | |
| // Unless required by applicable law or agreed to in writing, software | |
| // distributed under the License is distributed on an "AS IS" BASIS, | |
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | |
| // See the License for the specific language governing permissions and | |
| // limitations under the License. | |
| package com.google.gerrit.server.restapi.change; | |
| import static com.google.gerrit.extensions.conditions.BooleanCondition.and; | |
| import static com.google.gerrit.server.permissions.ChangePermission.ABANDON; | |
| import static com.google.gerrit.server.permissions.RefPermission.CREATE_CHANGE; | |
| import static com.google.gerrit.server.project.ProjectCache.illegalState; | |
| import static com.google.gerrit.server.query.change.ChangeData.asChanges; | |
| import static com.google.gerrit.server.update.context.RefUpdateContext.RefUpdateType.CHANGE_MODIFICATION; | |
| import com.google.common.base.Strings; | |
| import com.google.gerrit.common.Nullable; | |
| import com.google.gerrit.entities.BranchNameKey; | |
| import com.google.gerrit.entities.Change; | |
| import com.google.gerrit.entities.LabelType; | |
| import com.google.gerrit.entities.PatchSet; | |
| import com.google.gerrit.entities.PatchSetApproval; | |
| import com.google.gerrit.entities.Project; | |
| import com.google.gerrit.entities.RefNames; | |
| import com.google.gerrit.extensions.api.changes.MoveInput; | |
| import com.google.gerrit.extensions.common.ChangeInfo; | |
| import com.google.gerrit.extensions.restapi.AuthException; | |
| import com.google.gerrit.extensions.restapi.BadRequestException; | |
| import com.google.gerrit.extensions.restapi.MethodNotAllowedException; | |
| import com.google.gerrit.extensions.restapi.ResourceConflictException; | |
| import com.google.gerrit.extensions.restapi.Response; | |
| import com.google.gerrit.extensions.restapi.RestApiException; | |
| import com.google.gerrit.extensions.restapi.RestModifyView; | |
| import com.google.gerrit.extensions.webui.UiAction; | |
| import com.google.gerrit.server.ChangeMessagesUtil; | |
| import com.google.gerrit.server.ChangeUtil; | |
| import com.google.gerrit.server.IdentifiedUser; | |
| import com.google.gerrit.server.PatchSetUtil; | |
| import com.google.gerrit.server.approval.ApprovalsUtil; | |
| import com.google.gerrit.server.change.ChangeJson; | |
| import com.google.gerrit.server.change.ChangeResource; | |
| import com.google.gerrit.server.config.GerritServerConfig; | |
| import com.google.gerrit.server.git.GitRepositoryManager; | |
| import com.google.gerrit.server.notedb.ChangeUpdate; | |
| import com.google.gerrit.server.permissions.GlobalPermission; | |
| import com.google.gerrit.server.permissions.PermissionBackend; | |
| import com.google.gerrit.server.permissions.PermissionBackendException; | |
| import com.google.gerrit.server.project.ProjectCache; | |
| import com.google.gerrit.server.project.ProjectState; | |
| import com.google.gerrit.server.query.change.InternalChangeQuery; | |
| import com.google.gerrit.server.update.BatchUpdate; | |
| import com.google.gerrit.server.update.BatchUpdateOp; | |
| import com.google.gerrit.server.update.ChangeContext; | |
| import com.google.gerrit.server.update.UpdateException; | |
| import com.google.gerrit.server.update.context.RefUpdateContext; | |
| import com.google.gerrit.server.util.time.TimeUtil; | |
| import com.google.inject.Inject; | |
| import com.google.inject.Provider; | |
| import com.google.inject.Singleton; | |
| import java.io.IOException; | |
| import java.util.Optional; | |
| import org.eclipse.jgit.lib.Config; | |
| import org.eclipse.jgit.lib.ObjectId; | |
| import org.eclipse.jgit.lib.Repository; | |
| import org.eclipse.jgit.revwalk.RevCommit; | |
| import org.eclipse.jgit.revwalk.RevWalk; | |
| @Singleton | |
| public class Move implements RestModifyView<ChangeResource, MoveInput>, UiAction<ChangeResource> { | |
| private final PermissionBackend permissionBackend; | |
| private final BatchUpdate.Factory updateFactory; | |
| private final ChangeJson.Factory json; | |
| private final GitRepositoryManager repoManager; | |
| private final Provider<InternalChangeQuery> queryProvider; | |
| private final ChangeMessagesUtil cmUtil; | |
| private final PatchSetUtil psUtil; | |
| private final ApprovalsUtil approvalsUtil; | |
| private final ProjectCache projectCache; | |
| private final boolean moveEnabled; | |
| @Inject | |
| Move( | |
| PermissionBackend permissionBackend, | |
| BatchUpdate.Factory updateFactory, | |
| ChangeJson.Factory json, | |
| GitRepositoryManager repoManager, | |
| Provider<InternalChangeQuery> queryProvider, | |
| ChangeMessagesUtil cmUtil, | |
| PatchSetUtil psUtil, | |
| ApprovalsUtil approvalsUtil, | |
| ProjectCache projectCache, | |
| @GerritServerConfig Config gerritConfig) { | |
| this.permissionBackend = permissionBackend; | |
| this.updateFactory = updateFactory; | |
| this.json = json; | |
| this.repoManager = repoManager; | |
| this.queryProvider = queryProvider; | |
| this.cmUtil = cmUtil; | |
| this.psUtil = psUtil; | |
| this.approvalsUtil = approvalsUtil; | |
| this.projectCache = projectCache; | |
| this.moveEnabled = gerritConfig.getBoolean("change", null, "move", true); | |
| } | |
| @Override | |
| public Response<ChangeInfo> apply(ChangeResource rsrc, MoveInput input) | |
| throws RestApiException, UpdateException, PermissionBackendException, IOException { | |
| if (!moveEnabled) { | |
| // This will be removed with the above config once we reach consensus for the move change | |
| // behavior. See: https://issues.gerritcodereview.com/issues/40009784 | |
| throw new MethodNotAllowedException("move changes endpoint is disabled"); | |
| } | |
| Change change = rsrc.getChange(); | |
| Project.NameKey project = rsrc.getProject(); | |
| IdentifiedUser caller = rsrc.getUser().asIdentifiedUser(); | |
| if (input.destinationBranch == null) { | |
| throw new BadRequestException("destination branch is required"); | |
| } | |
| input.destinationBranch = RefNames.fullName(input.destinationBranch); | |
| if (!change.isNew()) { | |
| throw new ResourceConflictException("Change is " + ChangeUtil.status(change)); | |
| } | |
| BranchNameKey newDest = BranchNameKey.create(project, input.destinationBranch); | |
| if (change.getDest().equals(newDest)) { | |
| throw new ResourceConflictException("Change is already destined for the specified branch"); | |
| } | |
| // Not allowed to move if the current patch set is locked. | |
| psUtil.checkPatchSetNotLocked(rsrc.getNotes()); | |
| // Keeping all votes can be confusing in the context of the destination branch, see the | |
| // discussion in | |
| // https://gerrit-review.googlesource.com/c/gerrit/+/129171 | |
| // Only administrators are allowed to keep all labels at their own risk. | |
| if (input.keepAllVotes | |
| && !permissionBackend.user(caller).test(GlobalPermission.ADMINISTRATE_SERVER)) { | |
| throw new AuthException("move is not permitted with keepAllVotes option"); | |
| } | |
| // Move requires abandoning this change, and creating a new change. | |
| try { | |
| rsrc.permissions().check(ABANDON); | |
| permissionBackend.user(caller).ref(newDest).check(CREATE_CHANGE); | |
| } catch (AuthException denied) { | |
| throw new AuthException("move not permitted", denied); | |
| } | |
| projectCache.get(project).orElseThrow(illegalState(project)).checkStatePermitsWrite(); | |
| Op op = new Op(input); | |
| try (RefUpdateContext ctx = RefUpdateContext.open(CHANGE_MODIFICATION)) { | |
| try (BatchUpdate u = updateFactory.create(project, caller, TimeUtil.now())) { | |
| u.addOp(change.getId(), op); | |
| u.execute(); | |
| } | |
| } | |
| return Response.ok(json.noOptions().format(op.getChange())); | |
| } | |
| private class Op implements BatchUpdateOp { | |
| private final MoveInput input; | |
| private Change change; | |
| private BranchNameKey newDestKey; | |
| Op(MoveInput input) { | |
| this.input = input; | |
| } | |
| @Nullable | |
| public Change getChange() { | |
| return change; | |
| } | |
| @Override | |
| public boolean updateChange(ChangeContext ctx) throws ResourceConflictException, IOException { | |
| change = ctx.getChange(); | |
| if (!change.isNew()) { | |
| throw new ResourceConflictException("Change is " + ChangeUtil.status(change)); | |
| } | |
| Project.NameKey projectKey = change.getProject(); | |
| newDestKey = BranchNameKey.create(projectKey, input.destinationBranch); | |
| BranchNameKey changePrevDest = change.getDest(); | |
| if (changePrevDest.equals(newDestKey)) { | |
| throw new ResourceConflictException("Change is already destined for the specified branch"); | |
| } | |
| final PatchSet.Id patchSetId = change.currentPatchSetId(); | |
| try (Repository repo = repoManager.openRepository(projectKey); | |
| RevWalk revWalk = new RevWalk(repo)) { | |
| RevCommit currPatchsetRevCommit = | |
| revWalk.parseCommit(psUtil.current(ctx.getNotes()).commitId()); | |
| ObjectId refId = repo.resolve(input.destinationBranch); | |
| // Check if destination ref exists in project repo | |
| if (refId == null) { | |
| throw new ResourceConflictException( | |
| "Destination " + input.destinationBranch + " not found in the project"); | |
| } | |
| RevCommit refCommit = revWalk.parseCommit(refId); | |
| if (revWalk.isMergedInto(currPatchsetRevCommit, refCommit)) { | |
| throw new ResourceConflictException( | |
| "Current patchset revision is reachable from tip of " + input.destinationBranch); | |
| } | |
| } | |
| Change.Key changeKey = change.getKey(); | |
| if (!asChanges(queryProvider.get().setLimit(1).byBranchKey(newDestKey, changeKey)) | |
| .isEmpty()) { | |
| throw new ResourceConflictException( | |
| "Destination " | |
| + newDestKey.shortName() | |
| + " has a different change with same change key " | |
| + changeKey); | |
| } | |
| if (!change.currentPatchSetId().equals(patchSetId)) { | |
| throw new ResourceConflictException("Patch set is not current"); | |
| } | |
| PatchSet.Id psId = change.currentPatchSetId(); | |
| ChangeUpdate update = ctx.getUpdate(psId); | |
| update.setBranch(newDestKey.branch()); | |
| change.setDest(newDestKey); | |
| if (!input.keepAllVotes) { | |
| updateApprovals(ctx, update, psId, projectKey); | |
| } | |
| StringBuilder msgBuf = new StringBuilder(); | |
| msgBuf.append("Change destination moved from "); | |
| msgBuf.append(changePrevDest.shortName()); | |
| msgBuf.append(" to "); | |
| msgBuf.append(newDestKey.shortName()); | |
| if (!Strings.isNullOrEmpty(input.message)) { | |
| msgBuf.append("\n\n"); | |
| msgBuf.append(input.message); | |
| } | |
| cmUtil.setChangeMessage(ctx, msgBuf.toString(), ChangeMessagesUtil.TAG_MOVE); | |
| return true; | |
| } | |
| /** | |
| * We have a long discussion about how to deal with its votes after moving a change from one | |
| * branch to another. In the end, we think only keeping the veto votes is the best way since | |
| * it's simple for us and less confusing for our users. See the discussion in the following | |
| * proposal: https://gerrit-review.googlesource.com/c/gerrit/+/129171 | |
| */ | |
| private void updateApprovals( | |
| ChangeContext ctx, ChangeUpdate update, PatchSet.Id psId, Project.NameKey project) { | |
| for (PatchSetApproval psa : approvalsUtil.byPatchSet(ctx.getNotes(), psId)) { | |
| ProjectState projectState = projectCache.get(project).orElseThrow(illegalState(project)); | |
| Optional<LabelType> type = | |
| projectState.getLabelTypes(ctx.getNotes()).byLabel(psa.labelId()); | |
| // Only keep veto votes, defined as votes where: | |
| // 1- the label function allows minimum values to block submission. | |
| // 2- the vote holds the minimum value. | |
| if (!type.isPresent() | |
| || (type.get().isMaxNegative(psa) && type.get().getFunction().isBlock())) { | |
| continue; | |
| } | |
| // Remove votes from NoteDb. | |
| update.removeApprovalFor(psa.accountId(), psa.label()); | |
| } | |
| } | |
| } | |
| @Override | |
| public UiAction.Description getDescription(ChangeResource rsrc) throws IOException { | |
| UiAction.Description description = | |
| new UiAction.Description() | |
| .setLabel("Move Change") | |
| .setTitle("Move change to a different branch") | |
| .setVisible(false); | |
| if (!moveEnabled) { | |
| return description; | |
| } | |
| Change change = rsrc.getChange(); | |
| if (!change.isNew()) { | |
| return description; | |
| } | |
| if (!projectCache | |
| .get(rsrc.getProject()) | |
| .orElseThrow(illegalState(rsrc.getProject())) | |
| .statePermitsWrite()) { | |
| return description; | |
| } | |
| if (psUtil.isPatchSetLocked(rsrc.getNotes())) { | |
| return description; | |
| } | |
| return description.setVisible( | |
| and( | |
| permissionBackend.user(rsrc.getUser()).ref(change.getDest()).testCond(CREATE_CHANGE), | |
| rsrc.permissions().testCond(ABANDON))); | |
| } | |
| } |