opendev/gerrit
Code Issues Proposed changes

Merge "Remove unneeded null checks"

Browse Source
This commit is contained in:
Shawn Pearce
2013-08-22 15:56:08 +00:00
committed by Gerrit Code Review
parent 25b45a8fe3 ad3e8bd921
commit d5943fbf00
4 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -280,7 +280,7 @@ public class PermissionEditor extends Composite implements Editor<Permission>,
validRange = null;
}
if (value != null && Permission.OWNER.equals(value.getName())) {
if (Permission.OWNER.equals(value.getName())) {
exclusiveGroup.setEnabled(false);
} else {
exclusiveGroup.setEnabled(!readOnly);

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

@@ -120,7 +120,7 @@ public class ProjectAccessEditor extends Composite implements
history.getStyle().setDisplay(Display.NONE);
}
addSection.setVisible(value != null && editing && (!value.getOwnerOf().isEmpty() || value.canUpload()));
addSection.setVisible(editing && (!value.getOwnerOf().isEmpty() || value.canUpload()));
}
@Override

4
gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/ChangeScreen.java
View File

@@ -339,13 +339,11 @@ public class ChangeScreen extends Screen
patchesList.addItem(Util.C.baseDiffItem());
}
for (PatchSet pId : detail.getPatchSets()) {
if (patchesList != null) {
patchesList.addItem(Util.M.patchSetHeader(pId.getPatchSetId()), pId
.getId().toString());
}
}
if (diffBaseId != null && patchesList != null) {
if (diffBaseId != null) {
patchesList.setSelectedIndex(diffBaseId.get());
}

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

@@ -418,7 +418,7 @@ public class ChangeUtil {
final GitReferenceUpdated gitRefUpdated, final ReviewDb db)
throws NoSuchChangeException, OrmException, IOException {
final PatchSet.Id patchSetId = patch.getId();
if (patch == null || !patch.isDraft()) {
if (!patch.isDraft()) {
throw new NoSuchChangeException(patchSetId.getParentKey());
}