opendev/gerrit
Code Issues Proposed changes

Revert: Don't reload change from database

Browse Source 
Just retrieve the change to revert from the change control.

Change-Id: Iabe7624c3dfda9a9c0ff32f7cc9ff561c6dd066c
Signed-off-by: Edwin Kempin <ekempin@google.com>
This commit is contained in:
Edwin Kempin
2016-02-17 13:40:12 +01:00
parent 42c9d52ab7
commit 4d755ef5f5
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/change/Revert.java
View File

@@ -134,7 +134,6 @@ public class Revert implements RestModifyView<ChangeResource, RevertInput>,
if (patch == null) {
throw new ResourceNotFoundException(changeIdToRevert.toString());
}
Change changeToRevert = db.get().changes().get(changeIdToRevert);
Project.NameKey project = ctl.getProject().getNameKey();
CurrentUser user = ctl.getUser();
@@ -160,6 +159,7 @@ public class Revert implements RestModifyView<ChangeResource, RevertInput>,
revertCommitBuilder.setAuthor(authorIdent);
revertCommitBuilder.setCommitter(authorIdent);
Change changeToRevert = ctl.getChange();
if (message == null) {
message = MessageFormat.format(
ChangeMessages.get().revertChangeDefaultMessage,