From dd7961bc443c732d0bbdd6bda47d6c2cfda5b290 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Tue, 17 May 2011 22:06:07 -0400 Subject: [PATCH] Building site content and documentation from Markdown. --- src/com/gitblit/wicket/pages/PatchPage.java | 22 ++++++++++++---------- 1 files changed, 12 insertions(+), 10 deletions(-) diff --git a/src/com/gitblit/wicket/pages/PatchPage.java b/src/com/gitblit/wicket/pages/PatchPage.java index 45c1bf5..62431d2 100644 --- a/src/com/gitblit/wicket/pages/PatchPage.java +++ b/src/com/gitblit/wicket/pages/PatchPage.java @@ -1,17 +1,15 @@ package com.gitblit.wicket.pages; -import javax.servlet.http.HttpServletRequest; - import org.apache.wicket.PageParameters; import org.apache.wicket.markup.html.WebPage; import org.apache.wicket.markup.html.basic.Label; -import org.apache.wicket.protocol.http.servlet.ServletWebRequest; import org.eclipse.jgit.lib.Repository; import org.eclipse.jgit.revwalk.RevCommit; import com.gitblit.GitBlit; import com.gitblit.utils.JGitUtils; import com.gitblit.utils.StringUtils; +import com.gitblit.wicket.GitBlitWebSession; import com.gitblit.wicket.WicketUtils; public class PatchPage extends WebPage { @@ -20,26 +18,30 @@ super(params); if (!params.containsKey("r")) { - error("Repository not specified!"); + GitBlitWebSession.get().cacheErrorMessage("Repository not specified!"); redirectToInterceptPage(new RepositoriesPage()); + return; } + final String repositoryName = WicketUtils.getRepositoryName(params); final String baseObjectId = WicketUtils.getBaseObjectId(params); final String objectId = WicketUtils.getObject(params); final String blobPath = WicketUtils.getPath(params); - ServletWebRequest servletWebRequest = (ServletWebRequest) getRequest(); - HttpServletRequest req = servletWebRequest.getHttpServletRequest(); - req.getServerName(); - - Repository r = GitBlit.self().getRepository(req, repositoryName); + Repository r = GitBlit.self().getRepository(repositoryName); if (r == null) { - error("Can not load repository " + repositoryName); + GitBlitWebSession.get().cacheErrorMessage("Can not load repository " + repositoryName); redirectToInterceptPage(new RepositoriesPage()); return; } RevCommit commit = JGitUtils.getCommit(r, objectId); + if (commit == null) { + GitBlitWebSession.get().cacheErrorMessage("Commit is null"); + redirectToInterceptPage(new RepositoriesPage()); + return; + } + String patch; if (StringUtils.isEmpty(baseObjectId)) { patch = JGitUtils.getCommitPatch(r, commit, blobPath); -- Gitblit v1.9.1