opendev/gerrit
Code Issues Proposed changes

Rebase: Don't swallow caught exception

Browse Source 
Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: Ie485ec39ae7acc2863823fefa39494475c302079
This commit is contained in:
Edwin Kempin
2019-08-22 10:10:15 +02:00
parent 895fdc7536
commit d265c9c9c7
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
java/com/google/gerrit/extensions/restapi/UnprocessableEntityException.java
View File

@@ -21,4 +21,8 @@ public class UnprocessableEntityException extends RestApiException {
public UnprocessableEntityException(String msg) { public UnprocessableEntityException(String msg) {
super(msg); super(msg);
} }
public UnprocessableEntityException(String msg, Throwable cause) {
super(msg, cause);
}
} }

2
java/com/google/gerrit/server/restapi/change/Rebase.java
View File

@@ -165,7 +165,7 @@ public class Rebase extends RetryingRestModifyView<RevisionResource, RebaseInput
} }
} catch (NoSuchChangeException e) { } catch (NoSuchChangeException e) {
throw new UnprocessableEntityException( throw new UnprocessableEntityException(
String.format("Base change not found: %s", input.base)); String.format("Base change not found: %s", input.base), e);
} }
PatchSet.Id baseId = base.patchSet().id(); PatchSet.Id baseId = base.patchSet().id();