From 2a7306a1d92522569a8bb6e5a7c0bcdd5cf4cfaa Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Thu, 26 May 2011 17:11:38 -0400 Subject: [PATCH] Findbugs. CodePro Audit. Checkstyle. Unit test refactoring. --- src/com/gitblit/wicket/pages/PatchPage.java | 14 ++++++-------- 1 files changed, 6 insertions(+), 8 deletions(-) diff --git a/src/com/gitblit/wicket/pages/PatchPage.java b/src/com/gitblit/wicket/pages/PatchPage.java index 9d032ca..02e4d38 100644 --- a/src/com/gitblit/wicket/pages/PatchPage.java +++ b/src/com/gitblit/wicket/pages/PatchPage.java @@ -37,7 +37,7 @@ redirectToInterceptPage(new RepositoriesPage()); return; } - + final String repositoryName = WicketUtils.getRepositoryName(params); final String baseObjectId = WicketUtils.getBaseObjectId(params); final String objectId = WicketUtils.getObject(params); @@ -56,14 +56,12 @@ redirectToInterceptPage(new RepositoriesPage()); return; } - - String patch; - if (StringUtils.isEmpty(baseObjectId)) { - patch = JGitUtils.getCommitPatch(r, commit, blobPath); - } else { - RevCommit baseCommit = JGitUtils.getCommit(r, baseObjectId); - patch = JGitUtils.getCommitPatch(r, baseCommit, commit, blobPath); + + RevCommit baseCommit = null; + if (!StringUtils.isEmpty(baseObjectId)) { + baseCommit = JGitUtils.getCommit(r, baseObjectId); } + String patch = JGitUtils.getCommitPatch(r, baseCommit, commit, blobPath); add(new Label("patchText", patch)); r.close(); } -- Gitblit v1.9.1