opendev/gerrit
Code Issues Proposed changes

Merge "Mark DeletePrivate input as @Nullable" into stable-2.16

Browse Source
This commit is contained in:
David Pursehouse
2019-02-08 09:05:53 +00:00
committed by Gerrit Code Review
parent 461a0563ea f843d8674e
commit 4ab0c351a2
2 changed files with 11 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
java/com/google/gerrit/server/restapi/change/DeletePrivate.java
View File

@@ -16,6 +16,7 @@ package com.google.gerrit.server.restapi.change;
import static com.google.gerrit.extensions.conditions.BooleanCondition.or; import static com.google.gerrit.extensions.conditions.BooleanCondition.or;
import com.google.gerrit.common.Nullable;
import com.google.gerrit.extensions.conditions.BooleanCondition; import com.google.gerrit.extensions.conditions.BooleanCondition;
import com.google.gerrit.extensions.restapi.AuthException; import com.google.gerrit.extensions.restapi.AuthException;
import com.google.gerrit.extensions.restapi.ResourceConflictException; import com.google.gerrit.extensions.restapi.ResourceConflictException;
@@ -59,7 +60,7 @@ public class DeletePrivate
@Override @Override
protected Response<String> applyImpl( protected Response<String> applyImpl(
BatchUpdate.Factory updateFactory, ChangeResource rsrc, SetPrivateOp.Input input) BatchUpdate.Factory updateFactory, ChangeResource rsrc, @Nullable SetPrivateOp.Input input)
throws RestApiException, UpdateException { throws RestApiException, UpdateException {
if (!canDeletePrivate(rsrc).value()) { if (!canDeletePrivate(rsrc).value()) {
throw new AuthException("not allowed to unmark private"); throw new AuthException("not allowed to unmark private");

17
java/com/google/gerrit/server/restapi/change/SetPrivateOp.java
View File

@@ -15,6 +15,7 @@
package com.google.gerrit.server.restapi.change; package com.google.gerrit.server.restapi.change;
import com.google.common.base.Strings; import com.google.common.base.Strings;
import com.google.gerrit.common.Nullable;
import com.google.gerrit.extensions.restapi.ResourceConflictException; import com.google.gerrit.extensions.restapi.ResourceConflictException;
import com.google.gerrit.reviewdb.client.Change; import com.google.gerrit.reviewdb.client.Change;
import com.google.gerrit.reviewdb.client.ChangeMessage; import com.google.gerrit.reviewdb.client.ChangeMessage;
@@ -43,14 +44,14 @@ public class SetPrivateOp implements BatchUpdateOp {
} }
public interface Factory { public interface Factory {
SetPrivateOp create(ChangeMessagesUtil cmUtil, boolean isPrivate, Input input); SetPrivateOp create(ChangeMessagesUtil cmUtil, boolean isPrivate, @Nullable Input input);
} }
private final ChangeMessagesUtil cmUtil;
private final PatchSetUtil psUtil;
private final boolean isPrivate;
private final Input input;
private final PrivateStateChanged privateStateChanged; private final PrivateStateChanged privateStateChanged;
private final PatchSetUtil psUtil;
private final ChangeMessagesUtil cmUtil;
private final boolean isPrivate;
@Nullable private final Input input;
private Change change; private Change change;
private PatchSet ps; private PatchSet ps;
@@ -61,12 +62,12 @@ public class SetPrivateOp implements BatchUpdateOp {
PatchSetUtil psUtil, PatchSetUtil psUtil,
@Assisted ChangeMessagesUtil cmUtil, @Assisted ChangeMessagesUtil cmUtil,
@Assisted boolean isPrivate, @Assisted boolean isPrivate,
@Assisted Input input) { @Assisted @Nullable Input input) {
this.cmUtil = cmUtil; this.privateStateChanged = privateStateChanged;
this.psUtil = psUtil; this.psUtil = psUtil;
this.cmUtil = cmUtil;
this.isPrivate = isPrivate; this.isPrivate = isPrivate;
this.input = input; this.input = input;
this.privateStateChanged = privateStateChanged;
} }
@Override @Override