opendev/gerrit
Code Issues Proposed changes

replaced ternary operator with &&

Browse Source 
Replacing the ternary operator simplifies the code and makes it more
closely match the line above.

Change-Id: Idcc447f4f927b08ee2d66b3361a93e1fd9c217cb
This commit is contained in:
Matt Baker
2013-11-28 21:43:15 -07:00
parent a791cc4fdc
commit c019803bee
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectAccessScreen.java
View File

@@ -313,7 +313,7 @@ public class ProjectAccessScreen extends ProjectScreen {
private void enable(boolean enabled) {
commitMessage.setEnabled(enabled);
commit.setEnabled(enabled && !access.getOwnerOf().isEmpty());
review.setEnabled(enabled ? access.canUpload() : false);
review.setEnabled(enabled && access.canUpload());
cancel1.setEnabled(enabled);
cancel2.setEnabled(enabled);
}