diff options
author | Jacek Centkowski <geminica.programs@gmail.com> | 2013-10-30 07:36:31 -0700 |
---|---|---|
committer | Jacek Centkowski <geminica.programs@gmail.com> | 2013-10-30 07:53:15 -0700 |
commit | 3400f408a54bb8fac239a8a11b6332522d7f5fc3 (patch) | |
tree | 80d285d63d94dc3183de83653daa68a94004e528 | |
parent | dd1ef1d11440650123a1783339cde5d8fdd267e6 (diff) |
Extend AddBranchResult with toString method implementation
When RpcAuditEvent is dispatched it contains servlet response
(AddBranchResult object). Meaningful toString method improves
readability in audit log.
Change-Id: I44c5f5a68126894b4aab7958f8a00d15974fec6f
Signed-off-by: Jacek Centkowski <jcentkowski@collab.net>
-rw-r--r-- | gerrit-common/src/main/java/com/google/gerrit/common/data/AddBranchResult.java | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/gerrit-common/src/main/java/com/google/gerrit/common/data/AddBranchResult.java b/gerrit-common/src/main/java/com/google/gerrit/common/data/AddBranchResult.java index 24e527dce8..6facf73a44 100644 --- a/gerrit-common/src/main/java/com/google/gerrit/common/data/AddBranchResult.java +++ b/gerrit-common/src/main/java/com/google/gerrit/common/data/AddBranchResult.java @@ -41,6 +41,19 @@ public class AddBranchResult { return error; } + @Override + public String toString() { + if (hasError()) { + return getError().toString(); + } + if (getListBranchesResult() != null) { + return "succeed, no repository: " + + getListBranchesResult().getNoRepository() + ", can add: " + + getListBranchesResult().getCanAdd(); + } + return "succeed"; + } + public static class Error { public static enum Type { /** The branch cannot be created because the given branch name is invalid. */ |