Unit testing. Removal of some unused code paths.

James Moger

2011-05-28 28d6b2a860740557bf93dd0f9a48d059379ed696

 src/com/gitblit/wicket/pages/PatchPage.java

@@ -1,3 +1,18 @@
/*

 * Copyright 2011 gitblit.com.

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *     http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

package com.gitblit.wicket.pages;



import org.apache.wicket.PageParameters;

@@ -7,8 +22,10 @@
import org.eclipse.jgit.revwalk.RevCommit;



import com.gitblit.GitBlit;

import com.gitblit.utils.DiffUtils;

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 {

@@ -17,9 +34,11 @@
      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);

@@ -27,19 +46,23 @@


      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);

      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);			
      if (commit == null) {

         GitBlitWebSession.get().cacheErrorMessage("Commit is null");

         redirectToInterceptPage(new RepositoriesPage());

         return;

      }



      RevCommit baseCommit = null;

      if (!StringUtils.isEmpty(baseObjectId)) {

         baseCommit = JGitUtils.getCommit(r, baseObjectId);

      }

      String patch = DiffUtils.getCommitPatch(r, baseCommit, commit, blobPath);

      add(new Label("patchText", patch));

      r.close();

   }