Eliminated unnecessary repository enumeration (issue 103)

James Moger

2012-08-17 2a81783f291805594d4b97da6830aee94292f8c8

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

@@ -64,6 +64,7 @@


   IModel<String> username = new Model<String>("");

   IModel<String> password = new Model<String>("");

   List<RepositoryModel> repositoryModels = new ArrayList<RepositoryModel>();



   public RootPage() {

      super();

@@ -224,6 +225,15 @@
      }

   }



   protected List<RepositoryModel> getRepositoryModels() {

      if (repositoryModels.isEmpty()) {

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

         List<RepositoryModel> repositories = GitBlit.self().getRepositoryModels(user);

         repositoryModels.addAll(repositories);

      }

      return repositoryModels;

   }



   protected void addDropDownMenus(List<PageRegistration> pages) {



   }

@@ -231,7 +241,7 @@
   protected List<DropDownMenuItem> getRepositoryFilterItems(PageParameters params) {

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

      Set<DropDownMenuItem> filters = new LinkedHashSet<DropDownMenuItem>();

      List<RepositoryModel> repositories = GitBlit.self().getRepositoryModels(user);

      List<RepositoryModel> repositories = getRepositoryModels();



      // accessible repositories by federation set

      Map<String, AtomicInteger> setMap = new HashMap<String, AtomicInteger>();

@@ -307,9 +317,8 @@
   }



   protected List<RepositoryModel> getRepositories(PageParameters params) {

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

      if (params == null) {

         return GitBlit.self().getRepositoryModels(user);

         return getRepositoryModels();

      }



      boolean hasParameter = false;

@@ -319,7 +328,7 @@
      String team = WicketUtils.getTeam(params);

      int daysBack = params.getInt("db", 0);



      List<RepositoryModel> availableModels = GitBlit.self().getRepositoryModels(user);

      List<RepositoryModel> availableModels = getRepositoryModels();

      Set<RepositoryModel> models = new HashSet<RepositoryModel>();



      if (!StringUtils.isEmpty(repositoryName)) {