You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ev...@apache.org on 2009/06/22 15:11:53 UTC

svn commit: r787230 - /continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/GroupSummaryAction.java

Author: evenisse
Date: Mon Jun 22 13:11:53 2009
New Revision: 787230

URL: http://svn.apache.org/viewvc?rev=787230&view=rev
Log:
[CONTINUUM-2273] Improve performance by reducing the quantity of requests

Modified:
    continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/GroupSummaryAction.java

Modified: continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/GroupSummaryAction.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/GroupSummaryAction.java?rev=787230&r1=787229&r2=787230&view=diff
==============================================================================
--- continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/GroupSummaryAction.java (original)
+++ continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/GroupSummaryAction.java Mon Jun 22 13:11:53 2009
@@ -51,7 +51,9 @@
     {
         groups = new ArrayList<GroupSummary>();
 
+        //TODO: Merge this two requests to one
         Collection<ProjectGroup> projectGroups = getContinuum().getAllProjectGroups();
+        Map<Integer, ProjectGroupSummary> summaries = getContinuum().getProjectsSummaryByGroups();
 
         for ( ProjectGroup projectGroup : projectGroups )
         {
@@ -66,7 +68,6 @@
                 groupModel.setName( projectGroup.getName() );
                 groupModel.setDescription( projectGroup.getDescription() );
 
-                Map<Integer, ProjectGroupSummary> summaries = getContinuum().getProjectsSummaryByGroups();
                 ProjectGroupSummary summary = summaries.get( projectGroup.getId() );
 
                 if ( summary != null )