Improve relative path determination using Java 7 Paths

James Moger

2014-10-10 a74ddc24545ec45d0bb82ca2bb8f628ffdaa9da3

 src/main/java/com/gitblit/wicket/pages/RepositoryPage.java

@@ -60,7 +60,6 @@
import com.gitblit.models.UserRepositoryPreferences;

import com.gitblit.servlet.PagesServlet;

import com.gitblit.servlet.SyndicationServlet;

import com.gitblit.tickets.TicketIndexer.Lucene;

import com.gitblit.utils.ArrayUtils;

import com.gitblit.utils.BugtraqProcessor;

import com.gitblit.utils.DeepCopier;

@@ -70,7 +69,6 @@
import com.gitblit.wicket.CacheControl;

import com.gitblit.wicket.GitBlitWebSession;

import com.gitblit.wicket.SessionlessForm;

import com.gitblit.wicket.TicketsUI;

import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.panels.LinkPanel;

import com.gitblit.wicket.panels.NavigationPanel;

@@ -110,7 +108,7 @@
         error(MessageFormat.format(getString("gb.repositoryNotSpecifiedFor"), getPageName()), true);

      }



      if (!getRepositoryModel().hasCommits) {

      if (!getRepositoryModel().hasCommits && getClass() != EmptyRepositoryPage.class) {

         throw new RestartResponseException(EmptyRepositoryPage.class, params);

      }



@@ -150,6 +148,16 @@
         }

      }



      showAdmin = false;

      if (app().settings().getBoolean(Keys.web.authenticateAdminPages, true)) {

         boolean allowAdmin = app().settings().getBoolean(Keys.web.allowAdministration, false);

         showAdmin = allowAdmin && GitBlitWebSession.get().canAdmin();

      } else {

         showAdmin = app().settings().getBoolean(Keys.web.allowAdministration, false);

      }

      isOwner = GitBlitWebSession.get().isLoggedIn()

            && (getRepositoryModel().isOwner(GitBlitWebSession.get().getUsername()));



      // register the available navigation links for this page and user

      List<NavLink> navLinks = registerNavLinks();



@@ -158,10 +166,10 @@
      add(navigationPanel);



      add(new ExternalLink("syndication", SyndicationServlet.asLink(getRequest()

            .getRelativePathPrefixToContextRoot(), repositoryName, null, 0)));

            .getRelativePathPrefixToContextRoot(), getRepositoryName(), null, 0)));



      // add floating search form

      SearchForm searchForm = new SearchForm("searchForm", repositoryName);

      SearchForm searchForm = new SearchForm("searchForm", getRepositoryName());

      add(searchForm);

      searchForm.setTranslatedAttributes();



@@ -183,33 +191,49 @@
   }



   private List<NavLink> registerNavLinks() {

      Repository r = getRepository();

      RepositoryModel model = getRepositoryModel();



      PageParameters params = null;

      PageParameters objectParams = null;

      if (!StringUtils.isEmpty(repositoryName)) {

         params = WicketUtils.newRepositoryParameter(repositoryName);

         params = WicketUtils.newRepositoryParameter(getRepositoryName());

         objectParams = params;



         // preserve the objectid iff the objectid directly (or indirectly) refers to a ref

         if (isCommitPage() && !StringUtils.isEmpty(objectId)) {

            RevCommit commit = JGitUtils.getCommit(r, objectId);

            if (commit != null) {

               String bestId = getBestCommitId(commit);

               if (!commit.getName().equals(bestId)) {

                  objectParams = WicketUtils.newObjectParameter(getRepositoryName(), bestId);

               }

            }

         }

      }

      List<NavLink> navLinks = new ArrayList<NavLink>();



      Repository r = getRepository();

      RepositoryModel model = getRepositoryModel();



      // standard links

      if (RefLogUtils.getRefLogBranch(r) == null) {

         navLinks.add(new PageNavLink("gb.summary", SummaryPage.class, params));

      } else {

         navLinks.add(new PageNavLink("gb.summary", SummaryPage.class, params));

//         pages.put("overview", new PageRegistration("gb.overview", OverviewPage.class, params));

         //         pages.put("overview", new PageRegistration("gb.overview", OverviewPage.class, params));

         navLinks.add(new PageNavLink("gb.reflog", ReflogPage.class, params));

      }

      navLinks.add(new PageNavLink("gb.commits", LogPage.class, params));

      navLinks.add(new PageNavLink("gb.tree", TreePage.class, params));

      if (app().tickets().isReady() && (app().tickets().isAcceptingNewTickets(getRepositoryModel()) || app().tickets().hasTickets(getRepositoryModel()))) {

         PageParameters tParams = new PageParameters(params);

         for (String state : TicketsUI.openStatii) {

            tParams.add(Lucene.status.name(), state);

         }



      if (!model.hasCommits) {

         return navLinks;

      }



      navLinks.add(new PageNavLink("gb.commits", LogPage.class, objectParams));

      navLinks.add(new PageNavLink("gb.tree", TreePage.class, objectParams));

      if (app().tickets().isReady() && (app().tickets().isAcceptingNewTickets(model) || app().tickets().hasTickets(model))) {

         PageParameters tParams = WicketUtils.newOpenTicketsParameter(getRepositoryName());

         navLinks.add(new PageNavLink("gb.tickets", TicketsPage.class, tParams));
      }
      navLinks.add(new PageNavLink("gb.docs", DocsPage.class, params, true));

      navLinks.add(new PageNavLink("gb.docs", DocsPage.class, objectParams, true));

      if (app().settings().getBoolean(Keys.web.allowForking, true)) {

         navLinks.add(new PageNavLink("gb.forks", ForksPage.class, params, true));

      }

@@ -219,7 +243,7 @@
      // per-repository extra navlinks

      if (JGitUtils.getPagesBranch(r) != null) {

         ExternalNavLink pagesLink = new ExternalNavLink("gb.pages", PagesServlet.asLink(

               getRequest().getRelativePathPrefixToContextRoot(), repositoryName, null), true);

               getRequest().getRelativePathPrefixToContextRoot(), getRepositoryName(), null), true);

         navLinks.add(pagesLink);

      }



@@ -234,16 +258,6 @@
         navLinks.addAll(ext.getNavLinks(user, model));

      }



      // Conditionally add edit link

      showAdmin = false;

      if (app().settings().getBoolean(Keys.web.authenticateAdminPages, true)) {

         boolean allowAdmin = app().settings().getBoolean(Keys.web.allowAdministration, false);

         showAdmin = allowAdmin && GitBlitWebSession.get().canAdmin();

      } else {

         showAdmin = app().settings().getBoolean(Keys.web.allowAdministration, false);

      }

      isOwner = GitBlitWebSession.get().isLoggedIn()

            && (model.isOwner(GitBlitWebSession.get().getUsername()));

      return navLinks;

   }



@@ -316,7 +330,7 @@
      }



      // (un)star link allows a user to star a repository

      if (user.isAuthenticated) {

      if (user.isAuthenticated && model.hasCommits) {

         PageParameters starParams = DeepCopier.copy(getPageParameters());

         starParams.put(PARAM_STAR, !user.getPreferences().isStarredRepository(model.name));

         String toggleStarUrl = getRequestCycle().urlFor(getClass(), starParams).toString();

@@ -343,7 +357,7 @@
      } else {

         String fork = app().repositories().getFork(user.username, model.name);

         boolean hasFork = fork != null;

         boolean canFork = user.canFork(model);

         boolean canFork = user.canFork(model) && model.hasCommits;



         if (hasFork || !canFork) {

            // user not allowed to fork or fork already exists or repo forbids forking

@@ -422,6 +436,10 @@
      return m;

   }



   protected String getRepositoryName() {

      return getRepositoryModel().name;

   }



   protected RevCommit getCommit() {

      RevCommit commit = JGitUtils.getCommit(r, objectId);

      if (commit == null) {

@@ -449,7 +467,7 @@
      // find first branch match

      for (RefModel ref : JGitUtils.getLocalBranches(r, false, -1)) {

         if (ref.getObjectId().getName().equals(id)) {

            return ref.getName();

            return Repository.shortenRefName(ref.getName());

         }

      }



@@ -546,7 +564,8 @@
      String html;

      switch (model.commitMessageRenderer) {

      case MARKDOWN:

         html = MessageFormat.format("<div class='commit_message'>{0}</div>", content);

         String safeContent = app().xssFilter().relaxed(content);

         html = MessageFormat.format("<div class='commit_message'>{0}</div>", safeContent);

         break;

      default:

         html = MessageFormat.format("<pre class='commit_message'>{0}</pre>", content);

@@ -556,6 +575,10 @@
   }



   protected abstract String getPageName();



   protected boolean isCommitPage() {

      return false;

   }



   protected Component createPersonPanel(String wicketId, PersonIdent identity,

         Constants.SearchType searchType) {

@@ -630,7 +653,7 @@
         r = null;

      }

      // setup page header and footer

      setupPage(repositoryName, "/ " + getPageName());

      setupPage(getRepositoryName(), "/ " + getPageName());

      super.onBeforeRender();

   }