Fixed double enumeration of repositories on dashboard page

James Moger

2013-06-12 ada1a326fa36ceb46922006f87d4cff0c49e071f

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

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


import org.apache.wicket.Component;

import org.apache.wicket.PageParameters;

import org.apache.wicket.behavior.SimpleAttributeModifier;

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

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

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

@@ -41,9 +42,12 @@
import org.eclipse.jgit.lib.PersonIdent;

import org.eclipse.jgit.lib.Repository;

import org.eclipse.jgit.revwalk.RevCommit;

import org.slf4j.Logger;

import org.slf4j.LoggerFactory;



import com.gitblit.Constants;

import com.gitblit.GitBlit;

import com.gitblit.GitBlitException;

import com.gitblit.Keys;

import com.gitblit.PagesServlet;

import com.gitblit.SyndicationServlet;

@@ -52,7 +56,9 @@
import com.gitblit.models.RepositoryModel;

import com.gitblit.models.SubmoduleModel;

import com.gitblit.models.UserModel;

import com.gitblit.models.UserRepositoryPreferences;

import com.gitblit.utils.ArrayUtils;

import com.gitblit.utils.DeepCopier;

import com.gitblit.utils.JGitUtils;

import com.gitblit.utils.PushLogUtils;

import com.gitblit.utils.StringUtils;

@@ -67,7 +73,11 @@
import com.gitblit.wicket.panels.RefsPanel;



public abstract class RepositoryPage extends RootPage {

	
   private final Logger logger = LoggerFactory.getLogger(getClass());



   private final String PARAM_STAR = "star";

	
   protected final String projectName;

   protected final String repositoryName;

   protected final String objectId;

@@ -104,7 +114,7 @@
      if (getRepositoryModel().isCollectingGarbage) {

         error(MessageFormat.format(getString("gb.busyCollectingGarbage"), getRepositoryModel().name), true);

      }



		
      if (objectId != null) {

         RefModel branch = null;

         if ((branch = JGitUtils.getBranch(getRepository(), objectId)) != null) {

@@ -117,6 +127,22 @@
                        branch.reference.getName());

            if (!canAccess) {

               error(getString("gb.accessDenied"), true);

            }

         }

      }

		
      if (params.containsKey(PARAM_STAR)) {

         // set starred state

         boolean star = params.getBoolean(PARAM_STAR);

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

         if (user != null && user.isAuthenticated) {

            UserRepositoryPreferences prefs = user.getPreferences().getRepositoryPreferences(getRepositoryModel().name);

            prefs.starred = star;

            try {

               GitBlit.self().updateUserModel(user.username, user, false);

            } catch (GitBlitException e) {

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

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

            }

         }

      }

@@ -166,24 +192,26 @@
      } else {

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

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

      }

         pages.put("pushes", new PageRegistration("gb.pushes", PushesPage.class, params));

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

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

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

      if (GitBlit.getBoolean(Keys.web.allowForking, true)) {

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

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

      }



      // conditional links

      // per-repository extra page links

      if (model.useTickets && TicgitUtils.getTicketsBranch(r) != null) {

         pages.put("tickets", new PageRegistration("gb.tickets", TicketsPage.class, params));

         pages.put("tickets", new PageRegistration("gb.tickets", TicketsPage.class, params, true));

      }

      if (model.showReadme || model.useDocs) {

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

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

      }

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

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

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

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

         pages.put("pages", pagesLink);

      }



@@ -258,6 +286,26 @@
         }

      }

      

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

      if (user.isAuthenticated) {

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

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

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

         if (user.getPreferences().isStarredRepository(model.name)) {

            // show unstar button

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

            addToolbarButton("unstarLink", "icon-star-empty", getString("gb.unstar"), toggleStarUrl);

         } else {

            // show star button

            addToolbarButton("starLink", "icon-star", getString("gb.star"), toggleStarUrl);

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

         }

      } else {

         // anonymous user

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

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

      }



      // fork controls

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

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

@@ -289,6 +337,16 @@
      }

      

      super.setupPage(repositoryName, pageName);

   }

	
   protected void addToolbarButton(String wicketId, String iconClass, String label, String url) {

      Fragment button = new Fragment(wicketId, "toolbarLinkFragment", this);

      Label icon = new Label("icon");

      WicketUtils.setCssClass(icon, iconClass);

      button.add(icon);

      button.add(new Label("label", label));

      button.add(new SimpleAttributeModifier("href", url));

      add(button);

   }



   protected void addSyndicationDiscoveryLink() {

@@ -332,13 +390,14 @@
      RevCommit commit = JGitUtils.getCommit(r, objectId);

      if (commit == null) {

         error(MessageFormat.format(getString("gb.failedToFindCommit"),

               objectId, repositoryName, getPageName()), true);

               objectId, repositoryName, getPageName()), null, LogPage.class,

               WicketUtils.newRepositoryParameter(repositoryName));

      }

      getSubmodules(commit);

      return commit;

   }

   

   private Map<String, SubmoduleModel> getSubmodules(RevCommit commit) {	
   protected Map<String, SubmoduleModel> getSubmodules(RevCommit commit) {	
      if (submodules == null) {

         submodules = new HashMap<String, SubmoduleModel>();

         for (SubmoduleModel model : JGitUtils.getSubmodules(r, commit.getTree())) {

@@ -349,7 +408,10 @@
   }

   

   protected SubmoduleModel getSubmodule(String path) {

      SubmoduleModel model = submodules.get(path);

      SubmoduleModel model = null;

      if (submodules != null) {

         model = submodules.get(path);

      }

      if (model == null) {

         // undefined submodule?!

         model = new SubmoduleModel(path.substring(path.lastIndexOf('/') + 1), path, path);