opendev/gerrit
Code Issues Proposed changes

Merge "Revert: Use BooleanCondition for #setVisible"

Browse Source
This commit is contained in:
Shawn Pearce
2017-08-16 17:59:16 +00:00
committed by Gerrit Code Review
parent 01de1fc23f 7890f85bed
commit 22d44c21e2
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -14,6 +14,7 @@
package com.google.gerrit.server.change;
import static com.google.gerrit.extensions.conditions.BooleanCondition.and;
import static com.google.gerrit.server.permissions.RefPermission.CREATE_CHANGE;
import com.google.common.base.Strings;
@@ -240,8 +241,9 @@ public class Revert extends RetryingRestModifyView<ChangeResource, RevertInput,
.setLabel("Revert")
.setTitle("Revert the change")
.setVisible(
change.getStatus() == Change.Status.MERGED
&& permissionBackend.user(user).ref(change.getDest()).testOrFalse(CREATE_CHANGE));
and(
change.getStatus() == Change.Status.MERGED,
permissionBackend.user(user).ref(change.getDest()).testCond(CREATE_CHANGE)));
}
private class NotifyOp implements BatchUpdateOp {