Merge branch 'performance-improvements' into 'master'

Performance improvements Fixes #1135

Merge branch 'performance-improvements' into 'master'
Performance improvements Fixes #1135
Dmitriy Zaporozhets
2 parents 34cf3c49 5a064b02
Showing 5 changed files with 33 additions and 11 deletions Show diff stats
app/assets/stylesheets/gl_bootstrap.scss
app/controllers/admin/groups_controller.rb
app/controllers/admin/projects_controller.rb
app/views/admin/groups/show.html.haml
app/views/admin/projects/show.html.haml
@@ -253,6 +253,12 @@ ul.breadcrumb {
     }
   }
+  .panel-footer {
+    .pagination {
+      margin: 0;
+    }
+  }
+
   &.panel-small {
     .panel-heading {
       padding: 6px 15px;
@@ -8,6 +8,8 @@ class Admin::GroupsController &lt; Admin::ApplicationController
   end
   def show
+    @members = @group.members.order("group_access DESC").page(params[:members_page]).per(30)
+    @projects = @group.projects.page(params[:projects_page]).per(30)
   end
   def new
@@ -17,6 +17,11 @@ class Admin::ProjectsController &lt; Admin::ApplicationController
   end
   def show
+    if @group
+      @group_members = @group.members.order("group_access DESC").page(params[:group_members_page]).per(30)
+    end
+
+    @project_members = @project.users_projects.page(params[:project_members_page]).per(30)
   end
   def transfer
@@ -31,11 +31,12 @@
     .panel.panel-default
       .panel-heading
-        Projects
-        %small
-          (#{@group.projects.count})
+        %h3.panel-title
+          Projects
+          %span.badge
+            #{@group.projects.count}
       %ul.well-list
-        - @group.projects.sort_by(&:name).each do |project|
+        - @projects.each do |project|
           %li
             %strong
               = link_to project.name_with_namespace, [:admin, project]
@@ -43,6 +44,8 @@
                 = repository_size(project)
             %span.pull-right.light
               %span.monospace= project.path_with_namespace + ".git"
+      .panel-footer
+        = paginate @projects, param_name: 'projects_page', theme: 'gitlab'
   .col-md-6
     .panel.panel-default
@@ -62,12 +65,12 @@
           = submit_tag 'Add users into group', class: "btn btn-create"
     .panel.panel-default
       .panel-heading
-        %strong #{@group.name}
-        Group Members
-        %small
-          (#{@group.users_groups.count})
+        %h3.panel-title
+          Members
+          %span.badge
+            #{@group.users_groups.count}
       %ul.well-list.group-users-list
-        - @group.users_groups.order('group_access DESC').each do |member|
+        - @members.each do |member|
           - user = member.user
           %li{class: dom_class(user)}
             .list-item-name
@@ -77,3 +80,5 @@
               = member.human_access
               = link_to group_users_group_path(@group, member), data: { confirm: remove_user_from_group_message(@group, user) }, method: :delete, remote: true, class: "btn-tiny btn btn-remove", title: 'Remove user from group' do
                 %i.icon-minus.icon-white
+      .panel-footer
+        = paginate @members, param_name: 'members_page', theme: 'gitlab'
@@ -100,8 +100,10 @@
             = link_to admin_group_path(@group), class: 'btn btn-small' do
               %i.icon-edit
         %ul.well-list
-          - @group.users_groups.order('group_access DESC').each do |member|
+          - @group_members.each do |member|
             = render 'users_groups/users_group', member: member, show_controls: false
+        .panel-footer
+          = paginate @group_members, param_name: 'group_members_page', theme: 'gitlab'
     .panel.panel-default
       .panel-heading
@@ -113,7 +115,7 @@
             %i.icon-edit
             Manage Access
       %ul.well-list.team_members
-        - @project.users_projects.each do |users_project|
+        - @project_members.each do |users_project|
           - user = users_project.user
           %li.users_project
             .list-item-name
@@ -126,3 +128,5 @@
                 %span.light= users_project.human_access
                 = link_to project_team_member_path(@project, user), data: { confirm: remove_from_project_team_message(@project, user)}, method: :delete, remote: true, class: "btn btn-small btn-remove" do
                   %i.icon-remove
+      .panel-footer
+        = paginate @project_members, param_name: 'project_members_page', theme: 'gitlab'
	@@ -253,6 +253,12 @@ ul.breadcrumb {		@@ -253,6 +253,12 @@ ul.breadcrumb {
253	}	253	}
254	}	254	}
255		255
		256	+ .panel-footer {
		257	+ .pagination {
		258	+ margin: 0;
		259	+ }
		260	+ }
		261	+
256	&.panel-small {	262	&.panel-small {
257	.panel-heading {	263	.panel-heading {
258	padding: 6px 15px;	264	padding: 6px 15px;
	@@ -8,6 +8,8 @@ class Admin::GroupsController < Admin::ApplicationController		@@ -8,6 +8,8 @@ class Admin::GroupsController < Admin::ApplicationController
8	end	8	end
9		9
10	def show	10	def show
		11	+ @members = @group.members.order("group_access DESC").page(params[:members_page]).per(30)
		12	+ @projects = @group.projects.page(params[:projects_page]).per(30)
11	end	13	end
12		14
13	def new	15	def new