Merge pull request #1035 from gitblit/1034-RepositoryPageLifecycle

Paul Martin

2016-04-01 2c545280196ded9c042c0a8fa564048338bc19de

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

@@ -21,7 +21,6 @@
import java.util.Arrays;

import java.util.Date;

import java.util.HashMap;

import java.util.LinkedHashMap;

import java.util.LinkedHashSet;

import java.util.List;

import java.util.Map;

@@ -29,6 +28,7 @@


import org.apache.wicket.Component;

import org.apache.wicket.PageParameters;

import org.apache.wicket.RestartResponseException;

import org.apache.wicket.behavior.SimpleAttributeModifier;

import org.apache.wicket.markup.html.basic.Label;

import org.apache.wicket.markup.html.form.DropDownChoice;

@@ -48,29 +48,33 @@
import com.gitblit.Constants;

import com.gitblit.GitBlitException;

import com.gitblit.Keys;

import com.gitblit.PagesServlet;

import com.gitblit.SyndicationServlet;

import com.gitblit.extensions.RepositoryNavLinkExtension;

import com.gitblit.models.NavLink;

import com.gitblit.models.NavLink.ExternalNavLink;

import com.gitblit.models.NavLink.PageNavLink;

import com.gitblit.models.ProjectModel;

import com.gitblit.models.RefModel;

import com.gitblit.models.RepositoryModel;

import com.gitblit.models.SubmoduleModel;

import com.gitblit.models.UserModel;

import com.gitblit.models.UserRepositoryPreferences;

import com.gitblit.servlet.PagesServlet;

import com.gitblit.servlet.SyndicationServlet;

import com.gitblit.utils.ArrayUtils;

import com.gitblit.utils.BugtraqProcessor;

import com.gitblit.utils.DeepCopier;

import com.gitblit.utils.JGitUtils;

import com.gitblit.utils.MessageProcessor;

import com.gitblit.utils.ModelUtils;

import com.gitblit.utils.RefLogUtils;

import com.gitblit.utils.StringUtils;

import com.gitblit.wicket.CacheControl;

import com.gitblit.wicket.GitBlitWebSession;

import com.gitblit.wicket.PageRegistration;

import com.gitblit.wicket.PageRegistration.OtherPageLink;

import com.gitblit.wicket.SessionlessForm;

import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.panels.LinkPanel;

import com.gitblit.wicket.panels.NavigationPanel;

import com.gitblit.wicket.panels.RefsPanel;

import com.google.common.base.Optional;



public abstract class RepositoryPage extends RootPage {



@@ -88,14 +92,13 @@


   private Map<String, SubmoduleModel> submodules;



   private final Map<String, PageRegistration> registeredPages;

   private boolean showAdmin;

   private boolean isOwner;



   public RepositoryPage(PageParameters params) {

      super(params);

      repositoryName = WicketUtils.getRepositoryName(params);

      String root =StringUtils.getFirstPathElement(repositoryName);

      String root = StringUtils.getFirstPathElement(repositoryName);

      if (StringUtils.isEmpty(root)) {

         projectName = app().settings().getString(Keys.web.repositoryRootGroupName, "main");

      } else {

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

      }



      if (!getRepositoryModel().hasCommits) {

         setResponsePage(EmptyRepositoryPage.class, params);

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

         throw new RestartResponseException(EmptyRepositoryPage.class, params);

      }



      if (getRepositoryModel().isCollectingGarbage) {

@@ -139,7 +142,7 @@
            UserRepositoryPreferences prefs = user.getPreferences().getRepositoryPreferences(getRepositoryModel().name);

            prefs.starred = star;

            try {

               app().gitblit().updateUserModel(user.username, user, false);

               app().gitblit().reviseUser(user.username, user);

            } catch (GitBlitException e) {

               logger.error("Failed to update user " + user.username, e);

               error(getString("gb.failedToUpdateUser"), false);

@@ -147,19 +150,28 @@
         }

      }



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

      registeredPages = registerPages();

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



      // standard page links

      List<PageRegistration> pages = new ArrayList<PageRegistration>(registeredPages.values());

      NavigationPanel navigationPanel = new NavigationPanel("repositoryNavPanel", getRepoNavPageClass(), pages);

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

      List<NavLink> navLinks = registerNavLinks();



      // standard navigation links

      NavigationPanel navigationPanel = new NavigationPanel("repositoryNavPanel", getRepoNavPageClass(), navLinks);

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



@@ -176,56 +188,79 @@
      return getClass();

   }



   protected MessageProcessor messageProcessor() {

      return new MessageProcessor(app().settings());

   protected BugtraqProcessor bugtraqProcessor() {

      return new BugtraqProcessor(app().settings());

   }



   private Map<String, PageRegistration> registerPages() {

      PageParameters params = null;

      if (!StringUtils.isEmpty(repositoryName)) {

         params = WicketUtils.newRepositoryParameter(repositoryName);

      }

      Map<String, PageRegistration> pages = new LinkedHashMap<String, PageRegistration>();



   private List<NavLink> registerNavLinks() {

      Repository r = getRepository();

      RepositoryModel model = getRepositoryModel();



      PageParameters params = null;

      PageParameters objectParams = null;

      if (!StringUtils.isEmpty(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>();





      // standard links

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

         pages.put("summary", new PageRegistration("gb.summary", SummaryPage.class, params));

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

      } else {

         pages.put("summary", new PageRegistration("gb.summary", SummaryPage.class, params));

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

         pages.put("reflog", new PageRegistration("gb.reflog", ReflogPage.class, params));

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

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

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

      }

      pages.put("commits", new PageRegistration("gb.commits", LogPage.class, params));

      pages.put("tree", new PageRegistration("gb.tree", TreePage.class, params));

      pages.put("docs", new PageRegistration("gb.docs", DocsPage.class, params, true));

      pages.put("compare", new PageRegistration("gb.compare", ComparePage.class, params, true));



      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, objectParams, true));

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

         pages.put("forks", new PageRegistration("gb.forks", ForksPage.class, params, true));

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

      }

      navLinks.add(new PageNavLink("gb.compare", ComparePage.class, params, true));


      // conditional links

      // per-repository extra page links

      // per-repository extra navlinks

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

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

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

         pages.put("pages", pagesLink);

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

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

         navLinks.add(pagesLink);

      }



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

      UserModel user = UserModel.ANONYMOUS;

      if (GitBlitWebSession.get().isLoggedIn()) {

         user = GitBlitWebSession.get().getUser();

      }

      isOwner = GitBlitWebSession.get().isLoggedIn()

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

                  .getUsername()));

      return pages;



      // add repository nav link extensions

      List<RepositoryNavLinkExtension> extensions = app().plugins().getExtensions(RepositoryNavLinkExtension.class);

      for (RepositoryNavLinkExtension ext : extensions) {

         navLinks.addAll(ext.getNavLinks(user, model));

      }



      return navLinks;

   }



   protected boolean allowForkControls() {

@@ -234,8 +269,15 @@


   @Override

   protected void setupPage(String repositoryName, String pageName) {

		
      //This method should only be called once in the page lifecycle.

      //However, it must be called after the constructor has run, hence not in onInitialize

      //It may be attempted to be called again if an info or error message is displayed.

      if (get("projectTitle") != null) { return; }

		
      String projectName = StringUtils.getFirstPathElement(repositoryName);

      ProjectModel project = app().projects().getProjectModel(projectName);



      if (project.isUserProject()) {

         // user-as-project

         add(new LinkPanel("projectTitle", null, project.getDisplayName(),

@@ -252,7 +294,6 @@
      }

      add(new LinkPanel("repositoryName", null, name, SummaryPage.class,

            WicketUtils.newRepositoryParameter(repositoryName)));

      add(new Label("pageName", pageName).setRenderBodyOnly(true));



      UserModel user = GitBlitWebSession.get().getUser();

      if (user == null) {

@@ -263,25 +304,38 @@
      RepositoryModel model = getRepositoryModel();

      if (StringUtils.isEmpty(model.originRepository)) {

         if (model.isMirror) {

            add(new Fragment("repoIcon", "mirrorIconFragment", this));

            Fragment mirrorFrag = new Fragment("originRepository", "mirrorFragment", this);

            Label lbl = new Label("originRepository", MessageFormat.format(getString("gb.mirrorOf"), "<b>" + model.origin + "</b>"));

            mirrorFrag.add(lbl.setEscapeModelStrings(false));

            add(mirrorFrag);

         } else {

            add(new Label("originRepository").setVisible(false));

            if (model.isBare) {

               add(new Fragment("repoIcon", "repoIconFragment", this));

            } else {

               add(new Fragment("repoIcon", "cloneIconFragment", this));

            }

            add(new Label("originRepository", Optional.of(model.description).or("")));

         }

      } else {

         RepositoryModel origin = app().repositories().getRepositoryModel(model.originRepository);

         if (origin == null) {

            // no origin repository

            add(new Label("originRepository").setVisible(false));

            // no origin repository, show description if available

            if (model.isBare) {

               add(new Fragment("repoIcon", "repoIconFragment", this));

            } else {

               add(new Fragment("repoIcon", "cloneIconFragment", this));

            }

            add(new Label("originRepository", Optional.of(model.description).or("")));

         } else if (!user.canView(origin)) {

            // show origin repository without link

            add(new Fragment("repoIcon", "forkIconFragment", this));

            Fragment forkFrag = new Fragment("originRepository", "originFragment", this);

            forkFrag.add(new Label("originRepository", StringUtils.stripDotGit(model.originRepository)));

            add(forkFrag);

         } else {

            // link to origin repository

            add(new Fragment("repoIcon", "forkIconFragment", this));

            Fragment forkFrag = new Fragment("originRepository", "originFragment", this);

            forkFrag.add(new LinkPanel("originRepository", null, StringUtils.stripDotGit(model.originRepository),

                  SummaryPage.class, WicketUtils.newRepositoryParameter(model.originRepository)));

@@ -289,8 +343,16 @@
         }

      }



      // new ticket button

      if (user.isAuthenticated && app().tickets().isAcceptingNewTickets(getRepositoryModel())) {

         String newTicketUrl = getRequestCycle().urlFor(NewTicketPage.class, WicketUtils.newRepositoryParameter(repositoryName)).toString();

         addToolbarButton("newTicketLink", "fa fa-ticket", getString("gb.new"), newTicketUrl);

      } else {

         add(new Label("newTicketLink").setVisible(false));

      }



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

@@ -310,14 +372,16 @@
      }



      // fork controls

      if (!allowForkControls() || user == null || !user.isAuthenticated) {

      if (!allowForkControls() || !user.isAuthenticated) {

         // must be logged-in to fork, hide all fork controls

         add(new ExternalLink("forkLink", "").setVisible(false));

         add(new ExternalLink("myForkLink", "").setVisible(false));

      } else {

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

         String userRepo = ModelUtils.getPersonalPath(user.username) + "/" + StringUtils.stripDotGit(StringUtils.getLastPathElement(model.name));

         boolean hasUserRepo = app().repositories().hasRepository(userRepo);

         boolean hasFork = fork != null;

         boolean canFork = user.canFork(model);

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



         if (hasFork || !canFork) {

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

@@ -396,6 +460,10 @@
      return m;

   }



   protected String getRepositoryName() {

      return getRepositoryModel().name;

   }



   protected RevCommit getCommit() {

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

      if (commit == null) {

@@ -423,7 +491,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());

         }

      }



@@ -516,11 +584,12 @@


   protected void addFullText(String wicketId, String text) {

      RepositoryModel model = getRepositoryModel();

      String content = messageProcessor().processCommitMessage(model, text);

      String content = bugtraqProcessor().processCommitMessage(r, model, text);

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

@@ -530,6 +599,10 @@
   }



   protected abstract String getPageName();



   protected boolean isCommitPage() {

      return false;

   }



   protected Component createPersonPanel(String wicketId, PersonIdent identity,

         Constants.SearchType searchType) {

@@ -596,6 +669,7 @@
      }

   }





   @Override

   protected void onBeforeRender() {

      // dispose of repository object

@@ -603,8 +677,10 @@
         r.close();

         r = null;

      }

		
      // setup page header and footer

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

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



      super.onBeforeRender();

   }