opendev/gerrit
Code Issues Proposed changes

Merge "DeleteChange: 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 3dcf459299 5b3fa16f4a
commit b5e4b56fce
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-server/src/main/java/com/google/gerrit/server/change/DeleteChange.java
View File

@@ -14,6 +14,8 @@
package com.google.gerrit.server.change;
import static com.google.gerrit.extensions.conditions.BooleanCondition.and;
import com.google.gerrit.common.TimeUtil;
import com.google.gerrit.extensions.restapi.AuthException;
import com.google.gerrit.extensions.restapi.MethodNotAllowedException;
@@ -100,7 +102,7 @@ public class DeleteChange extends RetryingRestModifyView<ChangeResource, Input,
return new UiAction.Description()
.setLabel("Delete")
.setTitle("Delete change " + rsrc.getId())
.setVisible(couldDeleteWhenIn(status) && perm.testOrFalse(ChangePermission.DELETE));
.setVisible(and(couldDeleteWhenIn(status), perm.testCond(ChangePermission.DELETE)));
}
private boolean couldDeleteWhenIn(Change.Status status) {